mirror of https://github.com/python/cpython
unpack_string(): avoid a compiler warning (about a real bug!) by
copying the result of fgetc() into an int variable before testing it for EOF.
This commit is contained in:
parent
65692578b7
commit
0b624f69b5
|
@ -305,6 +305,7 @@ unpack_string(LogReaderObject *self, PyObject **pvalue)
|
|||
int i;
|
||||
int len;
|
||||
int err;
|
||||
int ch;
|
||||
char *buf;
|
||||
|
||||
if ((err = unpack_packed_int(self, &len, 0)))
|
||||
|
@ -312,7 +313,9 @@ unpack_string(LogReaderObject *self, PyObject **pvalue)
|
|||
|
||||
buf = malloc(len);
|
||||
for (i=0; i < len; i++) {
|
||||
if ((buf[i] = fgetc(self->logfp)) == EOF) {
|
||||
ch = fgetc(self->logfp);
|
||||
buf[i] = ch;
|
||||
if (ch == EOF) {
|
||||
free(buf);
|
||||
return ERR_EOF;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue