commit e66b83f8adff138247a030335360cba9f58704c7
author Niels Möller <nisse@webrtc.org> Mon May 30 12:57:41 2022 +0200
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon May 30 13:05:03 2022 +0000
tree 2219f28a4ed90d85294dfb63ddc4bcb4504e8fa4
parent c85e4737404a1399b5d4ffd8a8eb4ac8cd77fcdd

Never pass a signed char to ctype macros like isdigit()

Bug: None
Change-Id: I451bb2c1f175a77aefbc8363009bf35a769fe941
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/264442
Reviewed-by: Tomas Gunnarsson <tommi@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#37037}

modules/audio_coding/test/PCMFile.cc

diff --git a/modules/audio_coding/test/PCMFile.cc b/modules/audio_coding/test/PCMFile.cc
index 659f968..9ccfa49 100644
--- a/modules/audio_coding/test/PCMFile.cc
+++ b/modules/audio_coding/test/PCMFile.cc
@@ -62,7 +62,9 @@
   int16_t n = 0;
 
   // Removing trailing spaces.
-  while ((isspace(tmp_name[n]) || iscntrl(tmp_name[n])) && (tmp_name[n] != 0) &&
+  while ((isspace(static_cast<unsigned char>(tmp_name[n])) ||
+          iscntrl(static_cast<unsigned char>(tmp_name[n]))) &&
+         (static_cast<unsigned char>(tmp_name[n]) != 0) &&
          (n < MAX_FILE_NAME_LENGTH_BYTE)) {
     n++;
   }
@@ -73,7 +75,9 @@
   // Removing trailing spaces.
   n = (int16_t)(strlen(tmp_name) - 1);
   if (n >= 0) {
-    while ((isspace(tmp_name[n]) || iscntrl(tmp_name[n])) && (n >= 0)) {
+    while ((isspace(static_cast<unsigned char>(tmp_name[n])) ||
+            iscntrl(static_cast<unsigned char>(tmp_name[n]))) &&
+           (n >= 0)) {
       n--;
     }
   }