Anonymous | Login | Signup for a new account | 11-10-2008 11:11 PST |
Main | My View | View Issues | Change Log | Docs |
Viewing Issue Simple Details [ Jump to Notes ] | [ View Advanced ] [ Issue History ] [ Print ] | ||||||||
ID | Category | Severity | Reproducibility | Date Submitted | Last Update | ||||
0001201 | [BusyBox] Other | major | always | 02-06-07 20:11 | 02-07-07 12:27 | ||||
Reporter | rockeychu | View Status | public | ||||||
Assigned To | BusyBox | ||||||||
Priority | normal | Resolution | fixed | ||||||
Status | closed | Product Version | svn | ||||||
Summary | 0001201: Misuses "#ifdef ENABLE_FEATURE_EXEC_PREFER_APPLETS" (with patches) | ||||||||
Description |
Since SVN r17733, it introduced a micro of ENABLE_FEATURE_EXEC_PREFER_APPLETS, but misused it. ENABLE_FEATURE_EXEC_PREFER_APPLETS is always defined, but value just as 0 or 1, so you should use "if" instead of "#ifdef". BTW, I got it while new Busybox_SVN can't boot my puppy linux (it use pivot_root & chroot in init script, and chroot can't exec sh -c "/bin/busybox init"). Patches as following: Index: include/libbb.h =================================================================== --- include/libbb.h (revision 17803) +++ include/libbb.h (working copy) @@ -561,7 +561,7 @@ char *find_execable(const char *filename); int exists_execable(const char *filename); -#ifdef ENABLE_FEATURE_EXEC_PREFER_APPLETS +#if ENABLE_FEATURE_EXEC_PREFER_APPLETS int bb_execvp(const char *file, char *const argv[]); #define BB_EXECVP(prog,cmd) bb_execvp(prog,cmd) #define BB_EXECLP(prog,cmd,...) \ Index: libbb/execable.c =================================================================== --- libbb/execable.c (revision 17803) +++ libbb/execable.c (working copy) @@ -60,7 +60,7 @@ return 0; } -#ifdef ENABLE_FEATURE_EXEC_PREFER_APPLETS +#if ENABLE_FEATURE_EXEC_PREFER_APPLETS /* just like the real execvp, but try to launch an applet named 'file' first */ int bb_execvp(const char *file, char *const argv[]) |
||||||||
Additional Information | |||||||||
Attached Files | |||||||||
|
Copyright © 2000 - 2006 Mantis Group |