Re: [PATCH 04/15] prlimit: add warning diagnostic when attempting to change prlimit's limits

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/16/2011 01:45 PM, Karel Zak wrote:
On Mon, Nov 14, 2011 at 02:48:11AM +0100, Bernhard Voelker wrote:
>  +		if (lims[i].modify) {
>  +			if (!pid)
>  +				warnx(_("setting limits of the prlimit process is pointless"));

  Well, I don't think that we want such warnings in our utils:-)  Let's
  inform users about errors, the rest should be described in man pages.

hmm, then I tend to make it an error: prlimit acts too similar to
ulimit, but the *big* difference - and therefore the *big* trap for
the user - is that ulimit is a shell builtin which affects the current
shell process whereas prlimit runs in it's own process and, as a result,
will just do not have an effect without a PID.

Have a nice day,
Berny
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux