diff options
| author | Lars Hjemli | 2010-04-08 00:48:36 +0200 |
|---|---|---|
| committer | Lars Hjemli | 2010-04-08 00:48:36 +0200 |
| commit | ce761fde0772236a11b41270f9254c515f724ad5 (patch) | |
| tree | 8a7922811e56727c1444a3a93deecef2d2c08841 /shared.c | |
| parent | 9bd3a7b1ea6c5863d7cba1b5352ce707f5d129a0 (diff) | |
| download | cgit-ce761fde0772236a11b41270f9254c515f724ad5.tar.gz cgit-ce761fde0772236a11b41270f9254c515f724ad5.tar.bz2 cgit-ce761fde0772236a11b41270f9254c515f724ad5.zip | |
shared.c: avoid memory leak during diff
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (limited to 'shared.c')
| -rw-r--r-- | shared.c | 8 |
1 files changed, 8 insertions, 0 deletions
| @@ -278,6 +278,10 @@ int cgit_diff_files(const unsigned char *old_sha1, | |||
| 278 | if ((file1.ptr && buffer_is_binary(file1.ptr, file1.size)) || | 278 | if ((file1.ptr && buffer_is_binary(file1.ptr, file1.size)) || |
| 279 | (file2.ptr && buffer_is_binary(file2.ptr, file2.size))) { | 279 | (file2.ptr && buffer_is_binary(file2.ptr, file2.size))) { |
| 280 | *binary = 1; | 280 | *binary = 1; |
| 281 | if (file1.size) | ||
| 282 | free(file1.ptr); | ||
| 283 | if (file2.size) | ||
| 284 | free(file2.ptr); | ||
| 281 | return 0; | 285 | return 0; |
| 282 | } | 286 | } |
| 283 | 287 | ||
| @@ -290,6 +294,10 @@ int cgit_diff_files(const unsigned char *old_sha1, | |||
| 290 | emit_cb.outf = filediff_cb; | 294 | emit_cb.outf = filediff_cb; |
| 291 | emit_cb.priv = fn; | 295 | emit_cb.priv = fn; |
| 292 | xdl_diff(&file1, &file2, &diff_params, &emit_params, &emit_cb); | 296 | xdl_diff(&file1, &file2, &diff_params, &emit_params, &emit_cb); |
| 297 | if (file1.size) | ||
| 298 | free(file1.ptr); | ||
| 299 | if (file2.size) | ||
| 300 | free(file2.ptr); | ||
| 293 | return 0; | 301 | return 0; |
| 294 | } | 302 | } |
| 295 | 303 | ||
