bb login.c

Denys Vlasenko vda.linux at googlemail.com
Sat Jan 31 21:50:17 UTC 2009


On Saturday 31 January 2009 18:37, walter harms wrote:
> Hi Denis,
> i have broken down my patch to login.c into several pieces (4).
> Shall i send the diff or the .c ?

Please send diffs

> can i ping about ionice again again ? It is simply and 2.6 only.

Quality of code is still low.

For example, analyze what happens with variable "pid".
It will end up uninitialized if -p is not given,
and you will set ioprio to some random process.

Did you test "ioprio -c 1 -n 1 sleep 999"? Did you notice
that prio is not set correctly because of the above?

I cleaned the applet up, tested and applied it to svn.
I am asking you to test your code better.
--
vda


More information about the busybox mailing list