On Fri, 2006-05-26 at 18:56 -0400, Jeff Moyer wrote:
> Guillaume.Rousse> Third, the following patches still apply, but I got no
> Guillaume.Rousse> clue about their usefulness: -
> 
> Guillaume.Rousse> autofs-4.1.0-hesiod-bind.patch
> 
> jmoyer> I'll do some digging on this one tomorrow.
> 
> OK, it took me a while longer than expected to get to this.  Sorry!
> 
> The hesiod resolve patch is wanted.  Actually, I think in v5 we can just
> get rid of the old calls to hes_resolve.
> 
> If you look at the implementation that currently is there in lookup_hesiod.c,
> it has some bugs.  It will free a pointer that the library will
> subsequently try to free, and it also leaks memory.  This is, in part, due to
> the horrible definition of the interface at the time, I believe.
> 
> At any rate, we should move to hesiod_init, hesiod_resolve, etc.  It's
> probably best to check that the hesiod library supports the new interfaces
> at configure time.  If not, just disable the building of the hesiod
> modules.
> 
> Ian, let me know what you think of the attached patch.

Looks good.

I'll check more closely today and merge it if I don't see any problems.

Thanks Jeff.


_______________________________________________
autofs mailing list
[email protected]
http://linux.kernel.org/mailman/listinfo/autofs

Reply via email to