diff options
| author | Arthur de Jong <arthur@arthurdejong.org> | 2007-03-04 21:03:06 +0100 |
|---|---|---|
| committer | Arthur de Jong <arthur@arthurdejong.org> | 2007-03-04 21:03:06 +0100 |
| commit | 48d51b66fac883fc8648fb6b9df487382b8addc2 (patch) | |
| tree | e85684c4247e33f12417a95989210df016a35109 /nss/group.c | |
| parent | 03fdef7f5c7667fd92232f6d055f98833bcfef19 (diff) | |
code improvements by making type casts explicit, flagging ignored return values, renames and flagging of parameters and some miscelanious improvements (thanks to gcc warnings, splint, rats and flawfinder)
git-svn-id: http://arthurdejong.org/svn/nss-pam-ldapd/nss-ldapd@265 ef36b2f9-881f-0410-afb5-c4e39611909c
Diffstat (limited to 'nss/group.c')
| -rw-r--r-- | nss/group.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/nss/group.c b/nss/group.c index 4e51f94..ebf09a9 100644 --- a/nss/group.c +++ b/nss/group.c @@ -50,12 +50,12 @@ static enum nss_status read_gids( FILE *fp,long int *start,long int *size, gid_t **groupsp,long int limit,int *errnop) { - int32_t res=NSLCD_RESULT_SUCCESS; + int32_t res=(int32_t)NSLCD_RESULT_SUCCESS; int32_t tmpint32,tmp2int32,tmp3int32; gid_t gid; int num=0; /* loop over results */ - while (res==NSLCD_RESULT_SUCCESS) + while (res==(int32_t)NSLCD_RESULT_SUCCESS) { /* skip group name */ SKIP_STRING(fp); @@ -77,7 +77,7 @@ static enum nss_status read_gids( READ_TYPE(fp,res,int32_t); } /* return the proper status code */ - return (res==NSLCD_RESULT_NOTFOUND)?NSS_STATUS_SUCCESS:nslcd2nss(res); + return (res==(int32_t)NSLCD_RESULT_NOTFOUND)?NSS_STATUS_SUCCESS:nslcd2nss(res); } #endif /* REENABLE_WHEN_WORKING */ |