Re: [PATCH 0/4] Fix various integer overflows
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:48:06
Looks trivially correct; thanks.
Re: [PATCH 0/4] Fix various integer overflows
3 messages, 3 authors, 2016-06-15 · open the first message on its own page
Re: [PATCH 0/4] Fix various integer overflows
From: Stephen R. van den Berg <hidden>
Date: 2016-06-15 22:48:06
Junio C Hamano wrote:
Looks trivially correct; thanks.
I'm just curious, but is this based on an actual bug which someone
experienced, or is this just based on mere theoretical code analysis?
--
Sincerely,
Stephen R. van den Berg.
"Am I paying for this abuse or is it extra?"
Re: [PATCH 0/4] Fix various integer overflows
From: Ilari Liusvaara <hidden>
Date: 2016-06-15 22:48:06
On Wed, Jan 27, 2010 at 09:59:52AM +0100, Stephen R. van den Berg wrote:
Junio C Hamano wrote:quoted
Looks trivially correct; thanks.I'm just curious, but is this based on an actual bug which someone experienced, or is this just based on mere theoretical code analysis?
Theoretical at first, but I did construct packfile that hits one of
those overflows (the one in patch_delta(), 32 bits only).
In real world, hitting this bug would require hitting exactly 2^32-1
byte file, and that is quite rare size for file.
And what can happen with them in real world git usage is different
than what can happen with them if packs are suitably manipulated
("transport streams" and bundles both contain packs in them).
-Ilari