]> granicus.if.org Git - python/commit
SF patch #474175 (Jay T Miller): file.readinto arg parsing bug
authorGuido van Rossum <guido@python.org>
Tue, 23 Oct 2001 21:25:24 +0000 (21:25 +0000)
committerGuido van Rossum <guido@python.org>
Tue, 23 Oct 2001 21:25:24 +0000 (21:25 +0000)
commit00ebd46dfc8623ac0bd436e5d0c47b53cac29fb9
treeb31c56f7e70af0d9b1459602a504192de8b9d6e7
parent0d429e8cdd98399b3c7eae55984463856de9281d
SF patch #474175 (Jay T Miller): file.readinto arg parsing bug

    The C-code in fileobject.readinto(buffer) which parses
    the arguments assumes that size_t is interchangeable
    with int:

    size_t ntodo, ndone, nnow;

    if (f->f_fp == NULL)
    return err_closed();
    if (!PyArg_Parse(args, "w#", &ptr, &ntodo))
    return NULL;

    This causes a problem on Alpha / Tru64 / OSF1 v5.1
    where size_t is a long and sizeof(long) != sizeof(int).

    The patch I'm proposing declares ntodo as an int.  An
    alternative might be to redefine w# to expect size_t.

[We can't change w# because there are probably third party modules
relying on it. GvR]
Misc/ACKS
Objects/fileobject.c