Re: [PATCH] don't expect inode name to be NUL-terminated (avoid read overrun)

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

 



Jeff Garzik wrote:
> On 09/10/2010 08:55 AM, Jim Meyering wrote:
>>
>> * server/msg.c (msg_get): Copy only name_len bytes, then NUL-terminate,
>> rather than using snprintf to copy up to and including nonexistent NUL.
>> ---
>>
>> valgrind exposed this.  The use of snprintf would have been
>> correct if the inode name buffer (following the struct raw_inode)
>> were NUL-terminated, but it is not.
>
> applied -- good catch
>
> out of curiosity, what is your patch base?
>
> We combined cld and chunkd into a single 'hail' pkg, and from the
> pathname, your patch was generated from the older cld pkg.  We'd like
> to find the source and replace cld/chunkd with 'hail'.
>
> F12?  F13?  rawhide?

Hi Jeff,

I was using the sources from here:

    git://git.kernel.org/pub/scm/daemon/cld/cld.git

>From your comment there must be a hail git repository.
Found it:

    http://git.kernel.org/?p=daemon/distsrv/hail.git;a=summary

FYI, when I searched for hail's git repository initially,
https://hail.wiki.kernel.org/ was inaccessible, so I found
the above in a presumably-old cache.
--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Fedora Clound]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux