Check return value of filelength()

Fix CID 33266: Negative loop bound

Check return value of filelength()
Fix CID 33266: Negative loop bound
8118291d · Rob Swindell · ac6313df · 8118291d
Commit 8118291d authored Mar 03, 2022 by Rob Swindell 💬
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

src/sbbs3/useredit.cpp src/sbbs3/useredit.cpp +4 -0

No files found.
--- a/src/sbbs3/useredit.cpp
+++ b/src/sbbs3/useredit.cpp
@@ -686,6 +686,10 @@ int sbbs_t::searchup(char *search,int usernum)
 		return(usernum);

 	flen=(long)filelength(file);
+	if(flen < 0) {
+		close(file);
+		return usernum;
+	}
 	lseek(file,(long)((long)usernum*U_LEN),0);

 	while((i*U_LEN)<=(ulong)flen) {