Print error and return NULL from getfmsg() upon ftell error

Caught by Coverity - ftell[o] can return negative.

Print error and return NULL from getfmsg() upon ftell error
Caught by Coverity - ftell[o] can return negative.
160606e4 · Rob Swindell · d8f994cd · 160606e4
Commit 160606e4 authored Feb 22, 2021 by Rob Swindell 💬
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

src/sbbs3/sbbsecho.c src/sbbs3/sbbsecho.c +4 -0

No files found.
--- a/src/sbbs3/sbbsecho.c
+++ b/src/sbbs3/sbbsecho.c
@@ -3224,6 +3224,10 @@ char* getfmsg(FILE* stream, ulong* outlen)

 	length=0L;
 	start=ftello(stream);						/* Beginning of Message */
+	if(start < 0) {
+		lprintf(LOG_ERR, "ERROR %d line %d getting file offset", errno, __LINE__);
+		return NULL;
+	}
 	while(1) {
 		ch=fgetc(stream);						/* Look for Terminating NULL */
 		if(ch==0 || ch==EOF)					/* Found end of message */