[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gluster-devel] Crashes with latest git code
From: |
Raghavendra Bhat |
Subject: |
Re: [Gluster-devel] Crashes with latest git code |
Date: |
Thu, 17 May 2012 05:46:20 -0400 (EDT) |
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" <address@hidden>
To: "Raghavendra Bhat" <address@hidden>
Cc: address@hidden
Sent: Thursday, May 17, 2012 2:48:29 PM
Subject: Re: [Gluster-devel] Crashes with latest git code
Raghavendra Bhat <address@hidden> 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
address@hidden