Re: [PATCH] add missing libtirpc lib to mount_nfs.so when TIRPC enabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2013-07-24 at 10:37 +0800, Lan Yixun (dlan) wrote:
> From: "Lan Yixun (dlan)" <dennis.yxun@xxxxxxxxx>
> 
> this is need when libtirpc is enabled, otherwise autofs fail to start
> due to it can't find symbol clnt_dg_create (which is provided by libtirpc)

Yeah, I use libtirpc all the time but that's something I haven't
noticed, probably because TIRPC_WORKAROUND will cause the library to be
dlopen()ed at start.

So while the change is needed and I'll apply it, I'm not sure why you're
seeing it since the library should be open and loaded already.

> 
> ---
> autofs error log from /var/log/message:
> 
> Jul 23 12:11:58 ofire automount[25699]: open_mount:244: parse(sun): cannot open mount module
> nfs (/usr/lib64/autofs/mount_nfs.so: undefined symbol: clnt_dg_create)
> Jul 23 12:11:58 ofire automount[25699]: lookup(file): failed to open parse context
> 
> without this patch, mount_nfs.so fail link to libtirpc.so
>  # ldd /usr/lib64/autofs/mount_nfs.so |grep tirpc
> 
> with this patch applied:
>  # ldd /usr/lib64/autofs/mount_nfs.so |grep tirpc
>         libtirpc.so.1 => /lib64/libtirpc.so.1 (0x00007f7d94920000)
> 
> Signed-off-by: Lan Yixun (dlan) <dennis.yxun@xxxxxxxxx>
> ---
>  modules/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/modules/Makefile b/modules/Makefile
> index 4bb1096..0fe8ae8 100644
> --- a/modules/Makefile
> +++ b/modules/Makefile
> @@ -116,6 +116,6 @@ lookup_ldap.so: lookup_ldap.c dclist.o base64.o $(SASL_OBJ)
>  
>  mount_nfs.so: mount_nfs.c replicated.o
>  	$(CC) $(SOLDFLAGS) $(CFLAGS) -o mount_nfs.so \
> -		mount_nfs.c replicated.o $(AUTOFS_LIB) $(LIBNSL)
> +		mount_nfs.c replicated.o $(AUTOFS_LIB) $(LIBNSL) $(TIRPCLIB)
>  	$(STRIP) mount_nfs.so
>  


--
To unsubscribe from this list: send the line "unsubscribe autofs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux Ext4]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux