pkarashchenko commented on a change in pull request #1044:
URL: 
https://github.com/apache/incubator-nuttx-apps/pull/1044#discussion_r820229139



##########
File path: system/termcurses/tcurses_vt100.c
##########
@@ -782,7 +782,7 @@ static int tcurses_vt100_move(FAR struct termcurses_s *dev, 
int type,
   FAR struct tcurses_vt100_s *priv;
   int   ret = -ENOSYS;
   int   fd;
-  char  str[16];
+  char  str[32];

Review comment:
       should we change `sprintf` to `snprintf`?

##########
File path: system/nxplayer/nxplayer.c
##########
@@ -1778,7 +1778,7 @@ static int nxplayer_playinternal(FAR struct nxplayer_s 
*pplayer,
   struct audio_caps_desc_s cap_desc;
   struct ap_buffer_info_s  buf_info;
 #ifdef CONFIG_NXPLAYER_INCLUDE_MEDIADIR
-  char                path[128];
+  char                path[256];

Review comment:
       ```suggestion
     char                path[PATH_MAX];
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to