[Bug libc/1939] New: int return instead of enum value in inet/getnetgrent_r.c

["heiko dot nardmann at secunet dot com" <sourceware-bugzilla at sourceware dot org>]

In the file inet/getnetgrent_r.c in function setup() at line 57 there is the 
following block: 
 
  else if (startp == (service_user *) -1)  
    /* No services at all.  */  
    return 1;  
  else  
 
Now I wonder what "1" stands for; according to nss/nss.h "1" stands for 
NSS_STATUS_SUCCESS. But NSS_STATUS_UNAVAIL seems to be the right thing IMHO. 
At least returning an integer value instead of enum value is not clean.

-- 
           Summary: int return instead of enum value in inet/getnetgrent_r.c
           Product: glibc
           Version: 2.3.5
            Status: NEW
          Severity: minor
          Priority: P2
         Component: libc
        AssignedTo: drepper at redhat dot com
        ReportedBy: heiko dot nardmann at secunet dot com
                CC: glibc-bugs at sources dot redhat dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=1939

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.