On 15/11/2018 08:24, Florian Weimer wrote: > * Adhemerval Zanella: > >>> + if (slen == 0) >>> + __chk_fail (); >> >> Maybe a __glibc_unlikely here? > > I don't think this is required for paths which lead to a noreturn > function. > > Thanks, > Florian > Right, so I ok with current change as is.