Janitorial duty...

fix some undefined behaviour with increments in C. In theory a
compiler could have produced complete crap for this code. (tridge).
Jeremy.
(This used to be commit 2b4335f06265940582f389f48dc4f87f452a2703)

1 files changed, 8 insertions, 4 deletions

diff --git a/source3/lib/util_str.c b/source3/lib/util_str.c
index 148181fddd..799bc64cc6 100644
--- a/source3/lib/util_str.c
+++ b/source3/lib/util_str.c
@@ -1037,8 +1037,10 @@ void strlower_m(char *s)
 	   supported multi-byte character sets are ascii-compatible
 	   (ie. they match for the first 128 chars) */
 
-	while (*s && !(((unsigned char)s[0]) & 0x7F))
-		*s++ = tolower((unsigned char)*s);
+	while (*s && !(((unsigned char)s[0]) & 0x7F)) {
+		*s = tolower((unsigned char)*s);
+		s++;
+	}
 
 	if (!*s)
 		return;
@@ -1074,8 +1076,10 @@ void strupper_m(char *s)
 	   supported multi-byte character sets are ascii-compatible
 	   (ie. they match for the first 128 chars) */
 
-	while (*s && !(((unsigned char)s[0]) & 0x7F))
-		*s++ = toupper((unsigned char)*s);
+	while (*s && !(((unsigned char)s[0]) & 0x7F)) {
+		*s = toupper((unsigned char)*s);
+		s++;
+	}
 
 	if (!*s)
 		return;