diff options
| author | Tim Peters <tim.peters@gmail.com> | 2000-09-19 08:54:13 +0000 |
|---|---|---|
| committer | Tim Peters <tim.peters@gmail.com> | 2000-09-19 08:54:13 +0000 |
| commit | eef7758652a386659dcb73cd413669d937c7d7c7 (patch) | |
| tree | 4d701c6a07f2e13cab68df6e5f558a50df9539c0 /Python/marshal.c | |
| parent | f2420be88f9a8ce15be0786e450727767c912cc5 (diff) | |
| download | cpython-eef7758652a386659dcb73cd413669d937c7d7c7.tar.gz | |
Obscure marshal fixes:
When reading a short, sign-extend on platforms where shorts are
bigger than 16 bits.
When reading a long, repair the unportable sign extension that was
being done for 64-bit machines (it assumed that signed right shift
sign-extends).
Diffstat (limited to 'Python/marshal.c')
| -rw-r--r-- | Python/marshal.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Python/marshal.c b/Python/marshal.c index 91d322bb26..f8953ceae4 100644 --- a/Python/marshal.c +++ b/Python/marshal.c @@ -307,7 +307,8 @@ r_short(RFILE *p) register short x; x = r_byte(p); x |= r_byte(p) << 8; - /* XXX If your short is > 16 bits, add sign-extension here!!! */ + /* Sign-extension, in case short greater than 16 bits */ + x |= -(x & 0x8000); return x; } @@ -330,8 +331,7 @@ r_long(RFILE *p) } #if SIZEOF_LONG > 4 /* Sign extension for 64-bit machines */ - x <<= (8*sizeof(long) - 32); - x >>= (8*sizeof(long) - 32); + x |= -(x & 0x80000000L); #endif return x; } @@ -342,7 +342,7 @@ r_long64(RFILE *p) register long x; x = r_long(p); #if SIZEOF_LONG > 4 - x = (x & 0xFFFFFFFF) | (r_long(p) << 32); + x = (x & 0xFFFFFFFFL) | (r_long(p) << 32); #else if (r_long(p) != 0) { PyObject *f = PySys_GetObject("stderr"); |