Skip to content

Commit 8afb282

Browse files
PhilipOakleydscho
authored andcommitted
hash-object: demonstrate a >4GB/LLP64 problem
On LLP64 systems, such as Windows, the size of `long`, `int`, etc. is only 32 bits (for backward compatibility). Git's use of `unsigned long` for file memory sizes in many places, rather than size_t, limits the handling of large files on LLP64 systems (commonly given as `>4GB`). Provide a minimum test for handling a >4GB file. The `hash-object` command, with the `--literally` and without `-w` option avoids writing the object, either loose or packed. This avoids the code paths hitting the `bigFileThreshold` config test code, the zlib code, and the pack code. Subsequent patches will walk the test's call chain, converting types to `size_t` (which is larger in LLP64 data models) where appropriate. Signed-off-by: Philip Oakley <[email protected]> Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 787bfe4 commit 8afb282

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

t/t1007-hash-object.sh

+11
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,9 @@ test_expect_success 'setup' '
5050
5151
example sha1:ddd3f836d3e3fbb7ae289aa9ae83536f76956399
5252
example sha256:b44fe1fe65589848253737db859bd490453510719d7424daab03daf0767b85ae
53+
54+
large5GB sha1:0be2be10a4c8764f32c4bf372a98edc731a4b204
55+
large5GB sha256:dc18ca621300c8d3cfa505a275641ebab00de189859e022a975056882d313e64
5356
EOF
5457
'
5558

@@ -249,4 +252,12 @@ test_expect_success '--literally with extra-long type' '
249252
echo example | git hash-object -t $t --literally --stdin
250253
'
251254

255+
test_expect_failure EXPENSIVE,SIZE_T_IS_64BIT,!LONG_IS_64BIT \
256+
'files over 4GB hash literally' '
257+
test-tool genzeros $((5*1024*1024*1024)) >big &&
258+
test_oid large5GB >expect &&
259+
git hash-object --stdin --literally <big >actual &&
260+
test_cmp expect actual
261+
'
262+
252263
test_done

0 commit comments

Comments
 (0)