mirror of
https://git.yoctoproject.org/poky
synced 2026-05-30 00:20:08 +00:00
libtasn1: Backport compiler warning fixes
These patches are backported from master to fix issues raised by clang compiler. (From OE-Core rev: 6e3ff002e1a24936acb20dd209ea758c065cc16a) Signed-off-by: Khem Raj <raj.khem@gmail.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
+28
@@ -0,0 +1,28 @@
|
|||||||
|
From 908e9fa4c1172f09e0e45420a403dc25ed0a466c Mon Sep 17 00:00:00 2001
|
||||||
|
From: Nikos Mavrogiannopoulos <nmav@redhat.com>
|
||||||
|
Date: Tue, 26 Jul 2016 08:45:33 +0200
|
||||||
|
Subject: [PATCH 1/4] configure: don't add -Werror to build flags
|
||||||
|
|
||||||
|
---
|
||||||
|
Upstream-Status: Backport
|
||||||
|
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||||
|
|
||||||
|
configure.ac | 2 --
|
||||||
|
1 file changed, 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/configure.ac b/configure.ac
|
||||||
|
index 7a14e04..066f5fe 100644
|
||||||
|
--- a/configure.ac
|
||||||
|
+++ b/configure.ac
|
||||||
|
@@ -70,8 +70,6 @@ AC_ARG_ENABLE([gcc-warnings],
|
||||||
|
)
|
||||||
|
|
||||||
|
if test "$gl_gcc_warnings" = yes; then
|
||||||
|
- gl_WARN_ADD([-Werror], [WERROR_CFLAGS])
|
||||||
|
-
|
||||||
|
nw="$nw -Wsystem-headers" # Don't let system headers trigger warnings
|
||||||
|
nw="$nw -Wc++-compat" # We don't care strongly about C++ compilers
|
||||||
|
nw="$nw -Wtraditional" # Warns on #elif which we use often
|
||||||
|
--
|
||||||
|
1.9.1
|
||||||
|
|
||||||
@@ -0,0 +1,28 @@
|
|||||||
|
From 3542c01618fcde83b29640ea2c60bfd2629ae0b7 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Nikos Mavrogiannopoulos <nmav@redhat.com>
|
||||||
|
Date: Tue, 26 Jul 2016 08:47:49 +0200
|
||||||
|
Subject: [PATCH 2/4] ASN.y: corrected compiler warning
|
||||||
|
|
||||||
|
---
|
||||||
|
Upstream-Status: Backport
|
||||||
|
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||||
|
|
||||||
|
lib/ASN1.y | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/lib/ASN1.y b/lib/ASN1.y
|
||||||
|
index 731415d..6db638f 100644
|
||||||
|
--- a/lib/ASN1.y
|
||||||
|
+++ b/lib/ASN1.y
|
||||||
|
@@ -621,7 +621,7 @@ _asn1_create_errorDescription (int error, char *error_desc)
|
||||||
|
case ASN1_NAME_TOO_LONG:
|
||||||
|
snprintf (error_desc, ASN1_MAX_ERROR_DESCRIPTION_SIZE,
|
||||||
|
"%s:%u: name too long (more than %u characters)", file_name,
|
||||||
|
- line_number, ASN1_MAX_NAME_SIZE);
|
||||||
|
+ line_number, (unsigned)ASN1_MAX_NAME_SIZE);
|
||||||
|
break;
|
||||||
|
case ASN1_IDENTIFIER_NOT_FOUND:
|
||||||
|
snprintf (error_desc, ASN1_MAX_ERROR_DESCRIPTION_SIZE,
|
||||||
|
--
|
||||||
|
1.9.1
|
||||||
|
|
||||||
+73
@@ -0,0 +1,73 @@
|
|||||||
|
From c8903aa27dc9de1d9efeed9d1f7894f1019548f7 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Nikos Mavrogiannopoulos <nmav@redhat.com>
|
||||||
|
Date: Tue, 26 Jul 2016 08:49:15 +0200
|
||||||
|
Subject: [PATCH 3/4] parser_aux: corrected potential null pointer dereferences
|
||||||
|
|
||||||
|
---
|
||||||
|
Upstream-Status: Backport
|
||||||
|
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||||
|
|
||||||
|
lib/parser_aux.c | 12 ++++++------
|
||||||
|
1 file changed, 6 insertions(+), 6 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/lib/parser_aux.c b/lib/parser_aux.c
|
||||||
|
index 2285b20..12ee16f 100644
|
||||||
|
--- a/lib/parser_aux.c
|
||||||
|
+++ b/lib/parser_aux.c
|
||||||
|
@@ -637,7 +637,7 @@ _asn1_change_integer_value (asn1_node node)
|
||||||
|
p = NULL;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
- if (p->right)
|
||||||
|
+ if (p && p->right)
|
||||||
|
{
|
||||||
|
p = p->right;
|
||||||
|
break;
|
||||||
|
@@ -753,7 +753,7 @@ _asn1_expand_object_id (asn1_node node)
|
||||||
|
|
||||||
|
if (move == RIGHT)
|
||||||
|
{
|
||||||
|
- if (p->right)
|
||||||
|
+ if (p && p->right)
|
||||||
|
p = p->right;
|
||||||
|
else
|
||||||
|
move = UP;
|
||||||
|
@@ -828,7 +828,7 @@ _asn1_expand_object_id (asn1_node node)
|
||||||
|
|
||||||
|
if (move == RIGHT)
|
||||||
|
{
|
||||||
|
- if (p->right)
|
||||||
|
+ if (p && p->right)
|
||||||
|
p = p->right;
|
||||||
|
else
|
||||||
|
move = UP;
|
||||||
|
@@ -898,7 +898,7 @@ _asn1_type_set_config (asn1_node node)
|
||||||
|
|
||||||
|
if (move == RIGHT)
|
||||||
|
{
|
||||||
|
- if (p->right)
|
||||||
|
+ if (p && p->right)
|
||||||
|
p = p->right;
|
||||||
|
else
|
||||||
|
move = UP;
|
||||||
|
@@ -1007,7 +1007,7 @@ _asn1_check_identifier (asn1_node node)
|
||||||
|
p = NULL;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
- if (p->right)
|
||||||
|
+ if (p && p->right)
|
||||||
|
{
|
||||||
|
p = p->right;
|
||||||
|
break;
|
||||||
|
@@ -1067,7 +1067,7 @@ _asn1_set_default_tag (asn1_node node)
|
||||||
|
p = NULL;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
- if (p->right)
|
||||||
|
+ if (p && p->right)
|
||||||
|
{
|
||||||
|
p = p->right;
|
||||||
|
break;
|
||||||
|
--
|
||||||
|
1.9.1
|
||||||
|
|
||||||
@@ -0,0 +1,56 @@
|
|||||||
|
From d647bb2fa1bd288a6ac02c18318f3cba2a34c3a0 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Nikos Mavrogiannopoulos <nmav@redhat.com>
|
||||||
|
Date: Tue, 26 Jul 2016 08:50:24 +0200
|
||||||
|
Subject: [PATCH 4/4] tools: eliminated compiler warnings
|
||||||
|
|
||||||
|
---
|
||||||
|
Upstream-Status: Backport
|
||||||
|
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
||||||
|
|
||||||
|
src/asn1Coding.c | 2 +-
|
||||||
|
src/asn1Decoding.c | 2 +-
|
||||||
|
src/asn1Parser.c | 2 +-
|
||||||
|
3 files changed, 3 insertions(+), 3 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/asn1Coding.c b/src/asn1Coding.c
|
||||||
|
index d4df593..b516bfe 100644
|
||||||
|
--- a/src/asn1Coding.c
|
||||||
|
+++ b/src/asn1Coding.c
|
||||||
|
@@ -188,7 +188,7 @@ main (int argc, char *argv[])
|
||||||
|
default:
|
||||||
|
fprintf (stderr,
|
||||||
|
"asn1Coding: ?? getopt returned character code Ox%x ??\n",
|
||||||
|
- option_result);
|
||||||
|
+ (unsigned)option_result);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
diff --git a/src/asn1Decoding.c b/src/asn1Decoding.c
|
||||||
|
index 078963e..20f91ac 100644
|
||||||
|
--- a/src/asn1Decoding.c
|
||||||
|
+++ b/src/asn1Decoding.c
|
||||||
|
@@ -131,7 +131,7 @@ main (int argc, char *argv[])
|
||||||
|
default:
|
||||||
|
fprintf (stderr,
|
||||||
|
"asn1Decoding: ?? getopt returned character code Ox%x ??\n",
|
||||||
|
- option_result);
|
||||||
|
+ (unsigned)option_result);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
diff --git a/src/asn1Parser.c b/src/asn1Parser.c
|
||||||
|
index 7a3ae67..475bfc9 100644
|
||||||
|
--- a/src/asn1Parser.c
|
||||||
|
+++ b/src/asn1Parser.c
|
||||||
|
@@ -139,7 +139,7 @@ main (int argc, char *argv[])
|
||||||
|
default:
|
||||||
|
fprintf (stderr,
|
||||||
|
"asn1Parser: ?? getopt returned character code Ox%x ??\n",
|
||||||
|
- option_result);
|
||||||
|
+ (unsigned)option_result);
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
|
--
|
||||||
|
1.9.1
|
||||||
|
|
||||||
@@ -10,6 +10,10 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
|
|||||||
|
|
||||||
SRC_URI = "${GNU_MIRROR}/libtasn1/libtasn1-${PV}.tar.gz \
|
SRC_URI = "${GNU_MIRROR}/libtasn1/libtasn1-${PV}.tar.gz \
|
||||||
file://dont-depend-on-help2man.patch \
|
file://dont-depend-on-help2man.patch \
|
||||||
|
file://0001-configure-don-t-add-Werror-to-build-flags.patch \
|
||||||
|
file://0002-ASN.y-corrected-compiler-warning.patch \
|
||||||
|
file://0003-parser_aux-corrected-potential-null-pointer-derefere.patch \
|
||||||
|
file://0004-tools-eliminated-compiler-warnings.patch \
|
||||||
"
|
"
|
||||||
|
|
||||||
SRC_URI[md5sum] = "3018d0f466a32b66dde41bb122e6cab6"
|
SRC_URI[md5sum] = "3018d0f466a32b66dde41bb122e6cab6"
|
||||||
|
|||||||
Reference in New Issue
Block a user