diff options
| author | Christian Engelmayer <cengelma@gmx.at> | 2014-04-28 04:04:32 +0200 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2014-04-28 04:18:34 +0200 |
| commit | 2ee41e62ba5b952e9d9fcba6f7079a0c608bb849 (patch) | |
| tree | f04675dda077b17953fd1360763f0f8b8fd74b8f /scripts/mod | |
| parent | word-at-a-time: avoid undefined behaviour in zero_bytemask macro (diff) | |
| download | linux-2ee41e62ba5b952e9d9fcba6f7079a0c608bb849.tar.xz linux-2ee41e62ba5b952e9d9fcba6f7079a0c608bb849.zip | |
modpost: Fix resource leak in read_dump()
Function read_dump() memory maps the input via grab_file(), but fails to call
the corresponding unmap function. Add the missing call to release_file().
Detected by Coverity: CID 1192419
Signed-off-by: Christian Engelmayer <cengelma@gmx.at>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'scripts/mod')
| -rw-r--r-- | scripts/mod/modpost.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 066355673930..ea3e2bdf1825 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -2113,8 +2113,10 @@ static void read_dump(const char *fname, unsigned int kernel) s->preloaded = 1; sym_update_crc(symname, mod, crc, export_no(export)); } + release_file(file, size); return; fail: + release_file(file, size); fatal("parse error in symbol dump file\n"); } |