In getxattr name is NULL means its equivalent listxattr. So args->name being
NULL is ok. Process was crashing because it tried to do strdup (actually strlen
in the gf_strdup) of the NULL pointer to a string.
On wire we will send it as a null string with namelen set to 0 and
protocol/server will understand it.
On client side:
req.name = (char *)args->name;
if (!req.name) {
req.name = "";
req.namelen = 0;
}
On server side:
if (args.namelen)
state->name = gf_strdup (args.name);
----- Original Message -----
From: "Emmanuel Dreyfus" <[email protected]>
To: "Raghavendra Bhat" <[email protected]>
Cc: [email protected]
Sent: Thursday, May 17, 2012 2:48:29 PM
Subject: Re: [Gluster-devel] Crashes with latest git code
Raghavendra Bhat <[email protected]> wrote:
> A bug has already been filed for this (822385) and patch has been sent for
> the review (http://review.gluster.com/#change,3353).
I looked at the patch, it does not fix the problem I reported:
args->name is still NULL.
--
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
[email protected]
_______________________________________________
Gluster-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/gluster-devel
