From: Rich Felker <dalias@aerifal.cx>
Date: Sat, 6 Jul 2019 21:47:43 +0000 (-0400)
Subject: fix inadvertent use of uninitialized variable in dladdr
X-Git-Tag: v1.1.23~11
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=9b83182069cc3b213277104a992e195982060146;p=oweals%2Fmusl.git

fix inadvertent use of uninitialized variable in dladdr

commit c8b49b2fbc7faa8bf065220f11963d76c8a2eb93 introduced code that
checked bestsym to determine whether a matching symbol was found, but
bestsym is uninitialized if not. instead use best, consistent with use
in the rest of the function.

simplified from bug report and patch by Cheng Liu.
---

diff --git a/ldso/dynlink.c b/ldso/dynlink.c
index db543c19..d1edb131 100644
--- a/ldso/dynlink.c
+++ b/ldso/dynlink.c
@@ -2217,7 +2217,7 @@ int dladdr(const void *addr_arg, Dl_info *info)
 		}
 	}
 
-	if (bestsym && besterr > bestsym->st_size-1) {
+	if (best && besterr > bestsym->st_size-1) {
 		best = 0;
 		bestsym = 0;
 	}