Re: index-pack died on pread

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Linus Torvalds <torvalds@...>
Cc: Michal Rokos <michal.rokos@...>, GIT <git@...>
Date: Monday, July 23, 2007 - 2:03 pm

On Mon, 23 Jul 2007, Linus Torvalds wrote:


I doubt it can be that.  pread() is always used on pack data that we 
already received and validated, and part of the validation is the final 
pack SHA1.  No code path leads to pread() before the final pack SHA1 is 
tested OK.

The only way for the received pack to be truncated and pread() to fail 
is if write_or_die() somehow failed to write the pack data without Git 
noticing.


Nicolas
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
index-pack died on pread, Michal Rokos, (Mon Jul 23, 8:52 am)
Re: index-pack died on pread, Linus Torvalds, (Mon Jul 23, 1:04 pm)
Re: index-pack died on pread, Robin Rosenberg, (Wed Jul 25, 7:15 pm)
Re: index-pack died on pread, Linus Torvalds, (Wed Jul 25, 7:44 pm)
Re: index-pack died on pread, Alex Riesen, (Thu Jul 26, 8:42 am)
Re: index-pack died on pread, Linus Torvalds, (Thu Jul 26, 12:13 pm)
Re: index-pack died on pread, Junio C Hamano, (Thu Jul 26, 11:43 pm)
Re: index-pack died on pread, Linus Torvalds, (Fri Jul 27, 1:36 am)
Re: index-pack died on pread, Nicolas Pitre, (Fri Jul 27, 9:38 am)
Re: index-pack died on pread, Tomash Brechko, (Fri Jul 27, 5:50 am)
Re: index-pack died on pread, Tomash Brechko, (Fri Jul 27, 6:33 am)
Re: index-pack died on pread, Alex Riesen, (Thu Jul 26, 12:51 pm)
Re: index-pack died on pread, Linus Torvalds, (Thu Jul 26, 2:02 pm)
Re: index-pack died on pread, Nicolas Pitre, (Mon Jul 23, 2:03 pm)
Re: index-pack died on pread, Alex Riesen, (Mon Jul 23, 11:32 am)
Re: index-pack died on pread, Michal Rokos, (Wed Jul 25, 4:07 pm)
Re: index-pack died on pread, Alex Riesen, (Wed Jul 25, 4:48 pm)