ash: add a fixme comment at run_nofork_applet
authorDenys Vlasenko <vda.linux@googlemail.com>
Thu, 3 Aug 2017 17:56:29 +0000 (19:56 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Thu, 3 Aug 2017 17:56:29 +0000 (19:56 +0200)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c

index b285e3d33b3f96dd3f5975a6d53f00437454f492..8c9f4adc6586ff8c7ebef577b57410d86442edb9 100644 (file)
@@ -9918,11 +9918,13 @@ evalcommand(union node *cmd, int flags)
                if (applet_no >= 0 && APPLET_IS_NOFORK(applet_no)) {
                        listsetvar(varlist.list, VEXPORT|VSTACK);
                        /* run <applet>_main() */
+//FIXME: do we need INT_OFF / INT_ON here?
+//wouldn't open files and allocations leak on ^C otherwise?
                        status = run_nofork_applet(applet_no, argv);
                        break;
                }
 #endif
-               /* Can we avoid forking off? For example, very last command
+               /* Can we avoid forking? For example, very last command
                 * in a script or a subshell does not need forking,
                 * we can just exec it.
                 */