[Samba] [PATCH] typo & type

Gerald (Jerry) Carter jerry at samba.org
Thu Feb 22 17:27:26 GMT 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

ZIGLIO, Frediano, VF-IT wrote:
> Very silly patch. I like the english joke :)
> 
> type:
> 
> Index: nsswitch/nss_info_template.c
> ===================================================================
> --- nsswitch/nss_info_template.c        (revision 21498)
> +++ nsswitch/nss_info_template.c        (working copy)
> @@ -42,7 +42,7 @@
>                                        char **homedir,
>                                        char **shell,
>                                        char **gecos,
> -                                      uint32 *gid )
> +                                      gid_t *gid )
>  {
>         if ( !homedir || !shell || !gecos )
>                 return NT_STATUS_INVALID_PARAMETER;

Frediano,  The only problem is that the gid is parsed as
a uint32 through the winbindd code (in the request structure).
Did you experience a problem due to this?  Or just
stating that the gid_t type is more approriate (which I agree
with but that isa larger change)

The main problem in using gid_t is that we need a data type
of known size on both 32bit and 64bit compiles so that for
example /lib32/libnss_winbind.so can talk to a 64bit winbindd
running on the saying system.



> Index: nsswitch/nss_info.c
> ===================================================================
> --- nsswitch/nss_info.c (revision 21498)
> +++ nsswitch/nss_info.c (working copy)
> @@ -136,7 +136,7 @@
>         struct nss_function_entry *nss_backend;
>         struct nss_domain_entry *nss_domain;
> 
> -       /* The "template" backend should alqays be registered as it
> +       /* The "template" backend should always be registered as it
>            is a static module */

Thanks.  Fixing this now.





cheers, jerry
=====================================================================
Samba                                    ------- http://www.samba.org
Centeris                         -----------  http://www.centeris.com
"What man is a man who does not make the world better?"      --Balian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.4 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFF3dJ+IR7qMdg1EfYRAnkXAJ4+As5xbcmhODHPClneMZunwEw4UQCePINn
cWG225hNUjjpTNgsL7SQ3TA=
=ng+F
-----END PGP SIGNATURE-----


More information about the samba mailing list