Don Zickus <dzickus@xxxxxxxxxx> writes: > Recent changes to is_multipart_boundary() caused git-mailinfo to segfault. > The reason was after handling the end of the boundary the code tried to look > for another boundary. Because the boundary list was empty, dereferencing > the pointer to the top of the boundary caused the program to go boom. > > The fix is to check to see if the list is empty and if so go on its merry > way instead of looking for another boundary. Hmm, at this point !*content_top means that we are at the outermost level and we have just seen --boundary-- which is the terminating one, haven't we? Shouldn't we be simply returning? -- 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
