This patch fixes the undefined (according to C) and erroneous (under

Digital Uni x with both DEC cc and gcc) behaviour of modifying an
lvalue on the left side an d then using it on the right side of an
assignment. Since this code modifies the
 dbname parameter, it was changing, for example, "dbname=template1"
 into "dbname =emplate1".

David Smith Programmer P
This commit is contained in:
Bruce Momjian 1998-12-13 02:50:20 +00:00
parent 5c431eb1f2
commit ecdf95f6d6

View File

@ -1,6 +1,6 @@
/*------------------------------------------------------- /*-------------------------------------------------------
* *
* $Id: Pg.xs,v 1.9 1998/09/27 19:12:23 mergl Exp $ * $Id: Pg.xs,v 1.10 1998/12/13 02:50:20 momjian Exp $
* *
* Copyright (c) 1997, 1998 Edmund Mergl * Copyright (c) 1997, 1998 Edmund Mergl
* *
@ -215,7 +215,8 @@ PQconnectdb(conninfo)
} }
} else { } else {
while (*ptr && *ptr != ' ' && *ptr != '\t') { while (*ptr && *ptr != ' ' && *ptr != '\t') {
*ptr++ = tolower(*ptr); *ptr = tolower(*ptr);
ptr++;
} }
} }
} }
@ -732,7 +733,8 @@ connectdb(conninfo)
} }
} else { } else {
while (*ptr && *ptr != ' ' && *ptr != '\t') { while (*ptr && *ptr != ' ' && *ptr != '\t') {
*ptr++ = tolower(*ptr); *ptr = tolower(*ptr);
ptr++;
} }
} }
} }