[Pkg-shadow-devel] Bug#276419: Overflated severity?

Nicolas François Nicolas François <nicolas.francois@centraliens.net>, 276419@bugs.debian.org
Wed, 22 Jun 2005 22:24:14 +0200


tags 276419 pending
thanks

Hello Helmut,

Thanks a lot for your patches, and sorry for the late reply (mails sent to
-quiet@bugs.debian.org are not sent to the mailing list).

On Thu, Mar 31, 2005 at 02:22:27AM +0200, Helmut Waitzmann wrote:
> Nicolas François' patch solves the commandline problem.
> 
> A second problem, which is related to the first (erroneous processing
> of the shell's arguments), but afaics is not solved by his patch:
> 
> I like to use su when I'm going to change and test the configuration of
> my login shell.  To see, what's going on, when doing a login I'd like t=
o
> start the command
> 
> $ su -- - "$LOGNAME" -x
> 
> in order to get a login shell that echoes the commands it's going to
> execute.
[...]
> The Problem lies at lines 209 to 210 of Nicolas François' patched ver=
sion
> of su.c:
> 
>   if (command || additional_args)
>     args[argno++] = "-c";
> 
> This patch (applied after Nicolas François' patch) should correct tha=
t:
[...commandline.2.patch...]

This patch is applied to the Debian's shadow repository and should appear
in the next release.

> I'd like to see the function 'elements' returning a value of type size_=
t
> rather than of type int.
> 
> The following patch reflects this.
[...size_t.patch...]

I've not applied this one because I didn't want to scatter the Debian
patch (we are trying to update Debain's shadow to the lattest upstream,
and the size of the patch matters).

Kind Regards,
-- 
Nekral