diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2017-08-03 19:56:29 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2017-08-03 19:56:29 +0200 |
commit | cfd392bea9f52539baae9be4833075e464075958 (patch) | |
tree | 877a1408e3be6fa135da89def1d6d4fde870eefa | |
parent | 19c9f31af17f2c34e93c9c322b5c546ffbcda6ad (diff) | |
download | busybox-w32-cfd392bea9f52539baae9be4833075e464075958.tar.gz busybox-w32-cfd392bea9f52539baae9be4833075e464075958.tar.bz2 busybox-w32-cfd392bea9f52539baae9be4833075e464075958.zip |
ash: add a fixme comment at run_nofork_applet
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | shell/ash.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/shell/ash.c b/shell/ash.c index b285e3d33..8c9f4adc6 100644 --- a/shell/ash.c +++ b/shell/ash.c | |||
@@ -9918,11 +9918,13 @@ evalcommand(union node *cmd, int flags) | |||
9918 | if (applet_no >= 0 && APPLET_IS_NOFORK(applet_no)) { | 9918 | if (applet_no >= 0 && APPLET_IS_NOFORK(applet_no)) { |
9919 | listsetvar(varlist.list, VEXPORT|VSTACK); | 9919 | listsetvar(varlist.list, VEXPORT|VSTACK); |
9920 | /* run <applet>_main() */ | 9920 | /* run <applet>_main() */ |
9921 | //FIXME: do we need INT_OFF / INT_ON here? | ||
9922 | //wouldn't open files and allocations leak on ^C otherwise? | ||
9921 | status = run_nofork_applet(applet_no, argv); | 9923 | status = run_nofork_applet(applet_no, argv); |
9922 | break; | 9924 | break; |
9923 | } | 9925 | } |
9924 | #endif | 9926 | #endif |
9925 | /* Can we avoid forking off? For example, very last command | 9927 | /* Can we avoid forking? For example, very last command |
9926 | * in a script or a subshell does not need forking, | 9928 | * in a script or a subshell does not need forking, |
9927 | * we can just exec it. | 9929 | * we can just exec it. |
9928 | */ | 9930 | */ |