On Tue, Aug 23, Tomas Mraz wrote:
On Tue, 2011-08-16 at 17:19 +0400, Dmitry V. Levin wrote:
> On Tue, Aug 16, 2011 at 03:07:22PM +0200, Tomas Mraz wrote:
> > Hello,
> > there are missing dereferences in the _pam_parse() function in pam_env
> > module. The attached patch fixes this. The bug was found by Robert
> > Scheck.
> > OK to commit?
>
> Isn't argv[0][9] a more traditional than proposed *(*argv+9) ?
I'd prefer (*argv)[9] in this case then.
OK to commit with this substituted?
yes.
--
Thorsten Kukuk, Project Manager/Release Manager SLES
SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg)