about summary refs log tree commit diff
path: root/nss
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2009-07-01 02:42:59 -0700
committerUlrich Drepper <drepper@redhat.com>2009-07-01 02:42:59 -0700
commit29ba9812bb86bf6da9702f4fbff5cdbb25846401 (patch)
tree80640cc07ed58cf497a77602a28a75b2504c0a17 /nss
parente6bd12ddf72412918fe9002a3e27ecc07775bd64 (diff)
downloadglibc-29ba9812bb86bf6da9702f4fbff5cdbb25846401.tar.gz
glibc-29ba9812bb86bf6da9702f4fbff5cdbb25846401.tar.xz
glibc-29ba9812bb86bf6da9702f4fbff5cdbb25846401.zip
Fix getnetbyaddr implementation.
There were two problems in the getnetbyaddr implementation.  The type
argument is pretty much useless since (almost) no input file contains
this information and the NSS backends make up the value they fill in
for the n_addrtype field.  Therefore we now declare that passing AF_UNSPEC
is always recognized.  Secondly, the files backend didn't compare the network
numbers with the correct endianess.

Also change getent to take advantage of the type parameter change.
Diffstat (limited to 'nss')
-rw-r--r--nss/getent.c2
-rw-r--r--nss/nss_files/files-network.c5
2 files changed, 4 insertions, 3 deletions
diff --git a/nss/getent.c b/nss/getent.c
index 3a9430fd66..3a482e140f 100644
--- a/nss/getent.c
+++ b/nss/getent.c
@@ -534,7 +534,7 @@ networks_keys (int number, char *key[])
   for (i = 0; i < number; ++i)
     {
       if (isdigit (key[i][0]))
-	net = getnetbyaddr (inet_addr (key[i]), AF_UNIX);
+	net = getnetbyaddr (inet_addr (key[i]), AF_UNSPEC);
       else
 	net = getnetbyname (key[i]);
 
diff --git a/nss/nss_files/files-network.c b/nss/nss_files/files-network.c
index 9f4a3e0324..064de5a143 100644
--- a/nss/nss_files/files-network.c
+++ b/nss/nss_files/files-network.c
@@ -1,5 +1,5 @@
 /* Networks file parser in nss_files module.
-   Copyright (C) 1996, 1997, 1998, 2000, 2001 Free Software Foundation, Inc.
+   Copyright (C) 1996-1998, 2000, 2001, 2009 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
@@ -81,7 +81,8 @@ DB_LOOKUP (netbyname, ,,
 
 DB_LOOKUP (netbyaddr, ,,
 	   {
-	     if (result->n_addrtype == type && result->n_net == net)
+	     if ((type == AF_UNSPEC || result->n_addrtype == type)
+		 && result->n_net == htonl (net))
 	       /* Bingo!  */
 	       break;
 	   }, uint32_t net, int type)