nxlooper nxplayer nxrecorder: fix coverity bug

readline might return EOF. Signed-off-by: jihandong <jihandong@xiaomi.com>
apache · Aug 16, 2022 · 6afc226 · 6afc226
1 parent de26582
commit 6afc226
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/system/nxlooper/nxlooper_main.c b/system/nxlooper/nxlooper_main.c
@@ -528,9 +528,9 @@ int main(int argc, FAR char *argv[])
       /* Read a line from the terminal */
 
       len = readline(buffer, sizeof(buffer), stdin, stdout);
-      buffer[len] = '\0';
       if (len > 0)
         {
+          buffer[len] = '\0';
           if (strncmp(buffer, "!", 1) != 0)
             {
               /* nxlooper command */

diff --git a/system/nxplayer/nxplayer_main.c b/system/nxplayer/nxplayer_main.c
@@ -771,9 +771,9 @@ int main(int argc, FAR char *argv[])
       /* Read a line from the terminal */
 
       len = readline(buffer, sizeof(buffer), stdin, stdout);
-      buffer[len] = '\0';
       if (len > 0)
         {
+          buffer[len] = '\0';
           if (strncmp(buffer, "!", 1) != 0)
             {
               /* nxplayer command */

diff --git a/system/nxrecorder/nxrecorder_main.c b/system/nxrecorder/nxrecorder_main.c
@@ -484,9 +484,9 @@ int main(int argc, FAR char *argv[])
       /* Read a line from the terminal */
 
       len = readline(buffer, sizeof(buffer), stdin, stdout);
-      buffer[len] = '\0';
       if (len > 0)
         {
+          buffer[len] = '\0';
           if (strncmp(buffer, "!", 1) != 0)
             {
               /* nxrecorder command */