[GitHub] [nuttx] davids5 commented on a diff in pull request #8911: asprintf: Fixed possible memory leak if print fails.

[via GitHub]

davids5 commented on code in PR #8911:
URL: https://github.com/apache/nuttx/pull/8911#discussion_r1150457885


##########
libs/libc/stdio/lib_vasprintf.c:
##########
@@ -119,14 +119,18 @@ int vasprintf(FAR char **ptr, FAR const IPTR char *fmt, 
va_list ap)
 #endif
 
   /* Return a pointer to the string to the caller.  NOTE: the memstream put()
-   * method has already added the NUL terminator to the end of the string
+   * method has already added the NULL terminator to the end of the string

Review Comment:
   Oldtimers remember TTY really being a Teletype terminal. Here is where `NUL` 
is from.
    
   
![image](https://user-images.githubusercontent.com/1945821/228224038-e2a4ce1d-f8be-4a24-9540-d9ddc7998f67.png)
   



-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org