diff options
author | Ron Yorston <rmy@pobox.com> | 2016-04-03 22:29:35 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2016-04-03 22:29:35 +0200 |
commit | b22061718db0111f9e7474f9b60aef02456ac070 (patch) | |
tree | 95d4fc9424e79c0557d3d695d7600dbab792f3c3 | |
parent | 6aab9928dec29855bcee21bce163e5fdf7144350 (diff) | |
download | busybox-b22061718db0111f9e7474f9b60aef02456ac070.tar.gz |
find_applet_by_name: loop index should be signed
The loop
for (j = ARRAY_SIZE(applet_nameofs)-1; j >= 0; j--) {
was intended to terminate when j goes negative, so j needs to be signed.
Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | libbb/appletlib.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libbb/appletlib.c b/libbb/appletlib.c index d798a2eac..de654f64c 100644 --- a/libbb/appletlib.c +++ b/libbb/appletlib.c @@ -141,7 +141,8 @@ void FAST_FUNC bb_show_usage(void) int FAST_FUNC find_applet_by_name(const char *name) { - unsigned i, j, max; + unsigned i, max; + int j; const char *p; /* The commented-out word-at-a-time code is ~40% faster, but +160 bytes. |