Alex Riesen <raa.lkml@xxxxxxxxx> writes: > Junio C Hamano, Thu, Aug 02, 2007 23:58:41 +0200: >> "Alex Riesen" <raa.lkml@xxxxxxxxx> writes: >> > + if (chdir(dir)) >> > + rel = NULL; > ... >> >> Shouldn't it die() instead, though? > > Dunno. Don't like dying. I do not understand your reasoning. Why is it better to use mysteriously truncated path, which may result in doing something the user did not ask you to, rather than saying "No, my temporary buffer is not equipped to handle such an insanely long pathname"? >> Consolidating two of your patches, would this be Ok? > > Yes, but you may consider replacing strncpy with strlcpy: > >> + memcpy(dir_buffer, dir, len - suffix_len); >> + dir_buffer[len - suffix_len] = '\0'; > > strlcpy(dir_buffer, dir, len - suffix_len + 1); Does that buy us that much? Before going to that codepath, we have made sure the result fits, haven't we? - 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
