Blame SOURCES/perl-5.31.2-locale.c-Stop-Coverity-warning.patch
|
 |
4dad76 |
From 21dce8f4eb9136875a886371016aa25788f5144f Mon Sep 17 00:00:00 2001
|
|
|
4dad76 |
From: Karl Williamson <khw@cpan.org>
|
|
|
4dad76 |
Date: Tue, 6 Aug 2019 21:29:22 -0600
|
|
|
4dad76 |
Subject: [PATCH] locale.c: Stop Coverity warning
|
|
|
4dad76 |
MIME-Version: 1.0
|
|
|
4dad76 |
Content-Type: text/plain; charset=UTF-8
|
|
|
4dad76 |
Content-Transfer-Encoding: 8bit
|
|
|
4dad76 |
|
|
|
4dad76 |
Coverity is right, so re-order these clauses. This code is executed
|
|
|
4dad76 |
only if some very strange error occurs.
|
|
|
4dad76 |
|
|
|
4dad76 |
Signed-off-by: Petr Písař <ppisar@redhat.com>
|
|
|
4dad76 |
---
|
|
|
4dad76 |
locale.c | 11 ++++++-----
|
|
|
4dad76 |
1 file changed, 6 insertions(+), 5 deletions(-)
|
|
|
4dad76 |
|
|
|
4dad76 |
diff --git a/locale.c b/locale.c
|
|
|
4dad76 |
index db83d993de..af7af60038 100644
|
|
|
4dad76 |
--- a/locale.c
|
|
|
4dad76 |
+++ b/locale.c
|
|
|
4dad76 |
@@ -4349,11 +4349,6 @@ Perl__mem_collxfrm(pTHX_ const char *input_string,
|
|
|
4dad76 |
return xbuf;
|
|
|
4dad76 |
|
|
|
4dad76 |
bad:
|
|
|
4dad76 |
- Safefree(xbuf);
|
|
|
4dad76 |
- if (s != input_string) {
|
|
|
4dad76 |
- Safefree(s);
|
|
|
4dad76 |
- }
|
|
|
4dad76 |
- *xlen = 0;
|
|
|
4dad76 |
|
|
|
4dad76 |
# ifdef DEBUGGING
|
|
|
4dad76 |
|
|
|
4dad76 |
@@ -4363,6 +4358,12 @@ Perl__mem_collxfrm(pTHX_ const char *input_string,
|
|
|
4dad76 |
|
|
|
4dad76 |
# endif
|
|
|
4dad76 |
|
|
|
4dad76 |
+ Safefree(xbuf);
|
|
|
4dad76 |
+ if (s != input_string) {
|
|
|
4dad76 |
+ Safefree(s);
|
|
|
4dad76 |
+ }
|
|
|
4dad76 |
+ *xlen = 0;
|
|
|
4dad76 |
+
|
|
|
4dad76 |
return NULL;
|
|
|
4dad76 |
}
|
|
|
4dad76 |
|
|
|
4dad76 |
--
|
|
|
4dad76 |
2.20.1
|
|
|
4dad76 |
|