[source] busybox: nslookup_lede: fix compatibility with v1.25

LEDE Commits lede-commits at lists.infradead.org
Thu Apr 27 01:44:41 PDT 2017


jow pushed a commit to source.git, branch master:
https://git.lede-project.org/f1e32854619c09a49d85f5066517e300130cbcd2

commit f1e32854619c09a49d85f5066517e300130cbcd2
Author: Jo-Philipp Wich <jo at mein.io>
AuthorDate: Tue Apr 25 14:03:11 2017 +0200

    busybox: nslookup_lede: fix compatibility with v1.25
    
    The ":*" optstring syntax was only recently introduced with BusyBox v1.26,
    older versions need a corresponding hint in the "opt_complementary" variable
    to denote flag values that should be stored as llist entries.
    
    Add the required opt_complementary entry to fix random SIGBUS, SIGILL or
    SIGSEGV related crashes on BusyBox 1.25.x when attempting to use the "-q"
    flag of the "nslookup_lede" applet.
    
    Ref: https://forum.lede-project.org/t/nslookup-ipv6-in-lede-17-01-1
    
    Signed-off-by: Jo-Philipp Wich <jo at mein.io>
---
 package/utils/busybox/Makefile                            |  2 +-
 package/utils/busybox/patches/230-add_nslookup_lede.patch | 11 ++++++-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/package/utils/busybox/Makefile b/package/utils/busybox/Makefile
index c87243b..dd81045 100644
--- a/package/utils/busybox/Makefile
+++ b/package/utils/busybox/Makefile
@@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=busybox
 PKG_VERSION:=1.26.2
-PKG_RELEASE:=2
+PKG_RELEASE:=3
 PKG_FLAGS:=essential
 
 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.bz2
diff --git a/package/utils/busybox/patches/230-add_nslookup_lede.patch b/package/utils/busybox/patches/230-add_nslookup_lede.patch
index 51034e6..f37f4bd 100644
--- a/package/utils/busybox/patches/230-add_nslookup_lede.patch
+++ b/package/utils/busybox/patches/230-add_nslookup_lede.patch
@@ -1,4 +1,4 @@
-From 2285cd400a10b1bde5472fe1cec3311300ee943c Mon Sep 17 00:00:00 2001
+From 1188f159e0b0a9c07bed835c7313bedf5322a9e3 Mon Sep 17 00:00:00 2001
 From: Jo-Philipp Wich <jo at mein.io>
 Date: Tue, 14 Mar 2017 22:21:34 +0100
 Subject: [PATCH] networking: add LEDE nslookup applet
@@ -12,16 +12,16 @@ and the libresolv primitives to parse received DNS responses.
 
 Signed-off-by: Jo-Philipp Wich <jo at mein.io>
 ---
- networking/nslookup_lede.c | 893 +++++++++++++++++++++++++++++++++++++++++++++
- 1 file changed, 893 insertions(+)
+ networking/nslookup_lede.c | 894 +++++++++++++++++++++++++++++++++++++++++++++
+ 1 file changed, 894 insertions(+)
  create mode 100644 networking/nslookup_lede.c
 
 diff --git a/networking/nslookup_lede.c b/networking/nslookup_lede.c
 new file mode 100644
-index 000000000..6f1f86502
+index 000000000..f3e42d3bb
 --- /dev/null
 +++ b/networking/nslookup_lede.c
-@@ -0,0 +1,893 @@
+@@ -0,0 +1,894 @@
 +/*
 + * nslookup_lede - musl compatible replacement for busybox nslookup
 + *
@@ -760,6 +760,7 @@ index 000000000..6f1f86502
 +	applet_long_options = nslookup_longopts;
 +#endif
 +
++	opt_complementary = "q::";
 +	opts = getopt32(argv, "+q:*p:+r:+t:+s",
 +	                &type_strings, &default_port,
 +	                &default_retry, &default_timeout);



More information about the lede-commits mailing list