Patch from Florian Schirmer <jolt@tuxbox.org>. When I rewrote the parser,

I overlooked the case of someone wanting to use a valid but empty suid
config file.

diff --git a/applets/applets.c b/applets/applets.c
index f24679a1a38331e83fba9d54496aaf0f07f50958..8bb36dd6a5140223c23ca595ad286673d9b694f0 100644 (file)
--- a/applets/applets.c
+++ b/applets/applets.c
@@ -71,6 +71,7 @@ struct BB_suid_config
 };
 
 static struct BB_suid_config *suid_config;
+static int suid_cfg_readable;
 
 #endif /* CONFIG_FEATURE_SUID_CONFIG */
 
@@ -192,7 +193,7 @@ check_suid (struct BB_applet *applet)
   uid_t rgid = getgid ();
 
 #ifdef CONFIG_FEATURE_SUID_CONFIG
-  if (suid_config) {
+  if (suid_cfg_readable) {
        struct BB_suid_config *sct;
 
        for (sct = suid_config; sct; sct = sct->m_next) {
@@ -315,6 +316,7 @@ static void parse_config_file(void)
                return;
        }
 
+       suid_cfg_readable = 1;
        sct_head = NULL;
        section = lc = 0;