Am 2017-07-09 um 13:02 schrieb Nikolay Sivov: >> Checking 'dst' is not redundant though. >> > > Ah, maybe it is, because destination buffer length is never zero. Yeah, those functions want a non-NULL dst buffer if they have a dst length. if (!src || !srclen || (!dst && dstlen) || dstlen < 0)