Use (unsigned char) to make sure that only the control chars are

replaced with '?';

diff --git a/Zend/zend_exceptions.c b/Zend/zend_exceptions.c
index fed71cfac40d6774cee67587ee665848f825d4af..cf5a03824c6ba5ba78dedd130a268abc29559826 100644 (file)
--- a/Zend/zend_exceptions.c
+++ b/Zend/zend_exceptions.c
@@ -377,7 +377,7 @@ static int _build_trace_args(zval **arg, int num_args, va_list args, zend_hash_k
                                l_added += 3 + 1;
                        }
                        while (--l_added) {
-                               if ((*str)[*len - l_added] < 32) {
+                               if ((unsigned char)(*str)[*len - l_added] < 32) {
                                        (*str)[*len - l_added] = '?';
                                }
                        }
@@ -397,7 +397,7 @@ static int _build_trace_args(zval **arg, int num_args, va_list args, zend_hash_k
                                l_added += 3 + 1;
                        }
                        while (--l_added) {
-                               if ((*str)[*len - l_added] < 32) {
+                               if ((unsigned char)(*str)[*len - l_added] < 32) {
                                        (*str)[*len - l_added] = '?';
                                }
                        }