2008-05-05 15:16:30

by J. Bruce Fields

[permalink] [raw]
Subject: Re: [NFS] bug in nfs-utils support/misc/tcpwrapper.c

On Sat, May 03, 2008 at 01:49:20AM +0200, Sten Spans wrote:
>
> I've been trying to get /etc/netgroup support going on
> a machine serving nfs mounts to hosts located in various
> .nl netblocks. Managing hosts this way greatly simplifies
> hosts.allow and exports management, easily worth any slowdowns
> during mounting.
>
> I did notice a small problem in the tcpwrapper support:
> (resulting in errors from mountd)
>
> 127 /* Check the official name first. */
> 128 if (hosts_ctl(daemon, "", hp->h_name, ""))
> 129 return 1;
>
> this supplies the resolved hostname in the address field of hosts_ctl
> which doesn't work. Changing it to:
>
> if (hosts_ctl(daemon, hp->h_name, "", ""))
>
> does work as expected. The h_aliases check on line 133 probably needs
> the same change to work as expected.

Yep; looks like a clear bugfix. (If you have a moment to generate a
patch (with diff -u) showing both changes, that would probably help
Steve.)--b.

>
> --
> Sten Spans
>
> "There is a crack in everything, that's how the light gets in."
> Leonard Cohen - Anthem
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
> Don't miss this year's exciting event. There's still time to save $100.
> Use priority code J8TL2D2.
> http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
> _______________________________________________
> NFS maillist - [email protected]
> https://lists.sourceforge.net/lists/listinfo/nfs
> _______________________________________________
> Please note that [email protected] is being discontinued.
> Please subscribe to [email protected] instead.
> http://vger.kernel.org/vger-lists.html#linux-nfs
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs
_______________________________________________
Please note that [email protected] is being discontinued.
Please subscribe to [email protected] instead.
http://vger.kernel.org/vger-lists.html#linux-nfs



2008-05-05 15:42:19

by Sten Spans

[permalink] [raw]
Subject: Re: [NFS] bug in nfs-utils support/misc/tcpwrapper.c

_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs
_______________________________________________
Please note that [email protected] is being discontinued.
Please subscribe to [email protected] instead.
http://vger.kernel.org/vger-lists.html#linux-nfs


Attachments:
nfs-utils.diff (510.00 B)
(No filename) (320.00 B)
(No filename) (362.00 B)
Download all attachments