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