[Gluster-devel] NetBSD AFR regression

Justin Clift justin at gluster.org
Mon Mar 23 18:00:47 UTC 2015


Is there some way to put guards/checks in this bit of code to log
the callers + their arguments somewhere and then stand back to watch
it crash?  And figure it out from there?

Kinda guessing you guys are way past that though...

+ Justin


On 23 Mar 2015, at 17:39, Venky Shankar <yknev.shankar at gmail.com> wrote:
> There are other instances where "iobuf_arena->page_size" is used in
> iobuf.c and there are about a dozen callers for iobuf_size(). There
> needs to be something fishy about the call in the patch you mention to
> trigger the overrun.
> 
> 
> 
> On Mon, Mar 23, 2015 at 10:59 PM, Emmanuel Dreyfus <manu at netbsd.org> wrote:
>> On Mon, Mar 23, 2015 at 01:51:07PM +0100, Emmanuel Dreyfus wrote:
>>> I have ran out of smart ideas, and now I am going to start from older
>>> master and add commits to find the offending one. Any better idea is
>>> welcome.
>> 
>> git bissect points a finger to this change:
>> http://review.gluster.org/9708/
>> 
>> Inthe patch I find this:
>> +                iov.iov_len  = iobuf_size (iobuf)
>> 
>> iobuf_size() contains:
>>        size = iobuf->iobuf_arena->page_size;
>> 
>> and in iobuf_create_stdalloc_arena():
>>        iobuf_arena->page_size = 0x7fffffff;
>> 
>> Am I wrong, or is that a good recipe for an overrun?
>> 
>> 
>> 
>> --
>> Emmanuel Dreyfus
>> manu at netbsd.org
>> _______________________________________________
>> Gluster-devel mailing list
>> Gluster-devel at gluster.org
>> http://www.gluster.org/mailman/listinfo/gluster-devel
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at gluster.org
> http://www.gluster.org/mailman/listinfo/gluster-devel

--
GlusterFS - http://www.gluster.org

An open source, distributed file system scaling to several
petabytes, and handling thousands of clients.

My personal twitter: twitter.com/realjustinclift



More information about the Gluster-devel mailing list