Re: [patch] POSIX-compliant handling of getpwnam_r()/getgrnam_r() behavior

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

 



Andrei,
thanks for patch. Sadly, this part was changed in
52f88d04eaf2ad6c65df34a3401417d0583c6a45 (and I believe error you are
pointing out is still existing there). Can you please send a patch
applied on master git version?

Thanks,
  Honza

Andrei Belov napsal(a):
> Hello corosync users,
> 
> I've spent some time building and trying corosync 2.3.0 on SunOS
> (Joyent's SmartOS-based VM), and noticed one issue with uidgid {}
> configuration blocks parsing, e.g.:
> 
> uidgid {
>         uid: user|uid
>         gid: group|gid
> }
> 
> Corosync won't display any errors in case of non-existent username/uid
> or groupname/gid in configuration, and then we'll get something like
> that:
> 
> # corosync-cmapctl uidgid
> uidgid.gid.4096 (u8) = 1
> uidgid.uid.4292862832 (u8) = 1
> 
> I'm attaching the patch that fixes this issue by performing additional
> checks according to the following POSIX documents:
> 
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/getpwnam_r.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/getgrnam_r.html
> 
> [..]
> A null pointer is returned at the location pointed to by result on error or if the requested entry is not found.
> [..]
> 
> 
> Hope this would be helpful for someone.
> 
> Best regards,
> Andrei.
> 
> 
> 
> 
> 
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss




[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux