Hi Ditang- On Jun 8, 2014, at 9:42 PM, Ditang Chen <[email protected]> wrote:
> Can not retrieve FSN with the fsn_uuid when lookup a junction,the entries > is 1 and the 'tmp' pointer to NULL. > > # ls -l > total 8 > d--------T 2 root root 4096 Jun 4 07:05 s2 > # nsdb-list > NSDB: ldap.example.net:389 > NCE ou=fedfs,dc=example,dc=net has no FSN records > # nfsref lookup s2 > Segmentation fault > > (gdb) bt > #0 0x0000000000404eb0 in nfsref_lookup_resolve_fsn (host=0x118d630, > fsn_uuid=<optimized out>) at lookup.c:364 > #1 nfsref_lookup_nfs_fedfs (junct_path=<optimized out>) at lookup.c:483 > #2 0x0000000000403cf9 in main (argc=3, argv=0x7fff69dea5e8) at nfsref.c:181 Thanks, pulled this one for testing and further review. > Signed-off-by: [email protected] Just a nit: my git tool chain didn’t like your signed-off because it had only an email address. Can you use Signed-off-by: Ditang Chen <[email protected]> for future submissions? May I fix up this patch with the longer Signed-off-by: ? > --- > src/libnsdb/fileserver.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/src/libnsdb/fileserver.c b/src/libnsdb/fileserver.c > index 965f1a1..0899f00 100644 > --- a/src/libnsdb/fileserver.c > +++ b/src/libnsdb/fileserver.c > @@ -1527,8 +1527,14 @@ nsdb_get_fsn_find_entry_s(nsdb_t host, const char > *nce, const char *fsn_uuid, > ldap_msgfree(response); > > if (retval == FEDFS_OK) { > - xlog(D_CALL, "%s: returning fsn", __func__); > - *fsn = tmp; > + if (tmp == NULL) { > + xlog(D_CALL, "%s: No FSN entries for FSN UUID %s", > + __func__, fsn_uuid); > + retval = FEDFS_ERR_NSDB_NOFSN; > + } else { > + xlog(D_CALL, "%s: returning fsn", __func__); > + *fsn = tmp; > + } > } else > nsdb_free_fedfs_fsn(tmp); > return retval; > -- > 1.8.4.2 > -- Chuck Lever chuck[dot]lever[at]oracle[dot]com _______________________________________________ fedfs-utils-devel mailing list [email protected] https://oss.oracle.com/mailman/listinfo/fedfs-utils-devel
