shithub: choc

Info  •  Files  •  Log  •  Branches

doom: Don't ignore return value of fread() call

--- a/src/doom/m_menu.c

+++ b/src/doom/m_menu.c

@@ -510,6 +510,7 @@

     for (i = 0;i < load_end;i++)

     {

+        int retval;

         M_StringCopy(name, P_SaveGameFile(i), sizeof(name));

 	handle = fopen(name, "rb");

@@ -519,9 +520,9 @@

             LoadMenu[i].status = 0;

             continue;

         }

-	fread(&savegamestrings[i], 1, SAVESTRINGSIZE, handle);

+        retval = fread(&savegamestrings[i], 1, SAVESTRINGSIZE, handle);

 	fclose(handle);

-	LoadMenu[i].status = 1;

+        LoadMenu[i].status = retval == SAVESTRINGSIZE;

     }

 }