ulimit: set both hard and soft limits by default
authorAlexander Shishkin <virtuoso@slind.org>
Tue, 27 Jul 2010 06:40:55 +0000 (08:40 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Tue, 27 Jul 2010 06:40:55 +0000 (08:40 +0200)
function                                             old     new   delta
shell_builtin_ulimit                                 494     498      +4

Signed-off-by: Alexander Shishkin <virtuoso@slind.org>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/shell_common.c

index 3114ff3f7e897487ed2072122c4f24ede2c58ab0..dc363e298e5966b804d93068484fb9b8f610e72f 100644 (file)
@@ -428,9 +428,14 @@ shell_builtin_ulimit(char **argv)
                                                val <<= l->factor_shift;
                                        }
 //bb_error_msg("opt %c val_str:'%s' val:%lld", opt_char, val_str, (long long)val);
+                                       /* from man bash: "If neither -H nor -S
+                                        * is specified, both the soft and hard
+                                        * limits are set. */
+                                       if (!opts)
+                                               opts = OPT_hard + OPT_soft;
                                        if (opts & OPT_hard)
                                                limit.rlim_max = val;
-                                       if ((opts & OPT_soft) || opts == 0)
+                                       if (opts & OPT_soft)
                                                limit.rlim_cur = val;
 //bb_error_msg("setrlimit(%d, %lld, %lld)", l->cmd, (long long)limit.rlim_cur, (long long)limit.rlim_max);
                                        if (setrlimit(l->cmd, &limit) < 0) {