Untested patch for trivial memory leak in update-game-score.c

[Adam Richter <adamrichter4@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 632 bytes --]

Hi, everyone.

Attached is a patch to update-games-score.c for trivial memory leaks
found by cppcheck.  In the hopes of expediting integration, I hereby
release my interest in this patch into the public domain.

As far as I can tell, update-game-score.c is not compiled when I build
emacs, so the purpose of this patch might mostly be to clean up
cppcheck output, but I did compile my modified version of the file
manually, and gcc -Wall had no complaints, although gcc -Wextra
complained about some signed/unsigned comparisons and unlock_file()
ignoring its filename parameter.

Thanks in advance for considering this patch.

Adam

[-- Attachment #2: emacs.update-games-score.diff --]
[-- Type: text/x-diff, Size: 649 bytes --]

--- lib-src/update-game-score.c.orig	2019-01-07 06:26:06.000000000 -0800
+++ lib-src/update-game-score.c	2019-04-30 12:34:23.459986134 -0700
@@ -325,16 +325,21 @@ read_scores (const char *filename, struc
   nread = fread (filedata, 1, filesize + 1, f);
   if (filesize < nread)
     {
+      free (filedata);
       errno = 0;
       return -1;
     }
   if (nread < filesize)
     filesize = nread;
   if (ferror (f) || fclose (f) != 0)
-    return -1;
+    {
+      free (filedata);
+      return -1;
+    }
   filedata[filesize] = 0;
   if (strlen (filedata) != filesize)
     {
+      free (filedata);
       errno = 0;
       return -1;
     }