Issue #20947: Fixed a gcc warning with -Wstrict-overflow.

serhiy-storchaka · serhiy-storchaka · commit 5ae4f49f4a6a · 2016-09-27T22:03:51.000+03:00
diff --git a/Modules/_posixsubprocess.c b/Modules/_posixsubprocess.c
@@ -522,7 +522,7 @@ child_exec(char *const exec_array[],
         char *cur;
         _Py_write_noraise(errpipe_write, "OSError:", 8);
         cur = hex_errno + sizeof(hex_errno);
-        while (saved_errno != 0 && cur > hex_errno) {
+        while (saved_errno != 0 && cur != hex_errno) {
             *--cur = Py_hexdigits[saved_errno % 16];
             saved_errno /= 16;
         }

Original file line number	Diff line number	Diff line change
`@@ -522,7 +522,7 @@ child_exec(char *const exec_array[],`
`522`	`522`	`char *cur;`
`523`	`523`	`_Py_write_noraise(errpipe_write, "OSError:", 8);`
`524`	`524`	`cur = hex_errno + sizeof(hex_errno);`
`525`		`- while (saved_errno != 0 && cur > hex_errno) {`
	`525`	`+ while (saved_errno != 0 && cur != hex_errno) {`
`526`	`526`	`*--cur = Py_hexdigits[saved_errno % 16];`
`527`	`527`	`saved_errno /= 16;`
`528`	`528`	`}`