Pierre Habouzit <madcoder@xxxxxxxxxx> writes: > @@ -89,14 +86,13 @@ unsigned long interpolate(char *result, unsigned long reslen, > } > } > /* Straight copy one non-interpolation character. */ > + if (newlen < reslen) > *dest++ = *src; > src++; > newlen++; > } > > + if (reslen > 0) > + *dest = '\0'; > + return newlen; > } Are you sure about this *dest = '\0' assignment? Shouldn't it be done only when "(newlen < reslen)" just like the straight copy and substitution cases? - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html