diff options
| author | Thomas White <taw@bitwiz.org.uk> | 2011-04-13 22:41:29 +0200 |
|---|---|---|
| committer | Thomas White <taw@physics.org> | 2012-02-22 15:27:24 +0100 |
| commit | 17f331d526fa68e01e44c55f90a00bb7b9a3d637 (patch) | |
| tree | 7659452038632ea4792235ad4cd40d1425fc263c /src/mosflm.c | |
| parent | 2b01c35b2e7408b10b7958842ac07f0bb9cc1153 (diff) | |
Tidy up various memory leaks
Diffstat (limited to 'src/mosflm.c')
| -rw-r--r-- | src/mosflm.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/mosflm.c b/src/mosflm.c index b4906921..9ccbf0d5 100644 --- a/src/mosflm.c +++ b/src/mosflm.c @@ -505,7 +505,6 @@ static int mosflm_readable(struct image *image, struct mosflm_data *mosflm) void run_mosflm(struct image *image, UnitCell *cell) { - int fail; struct mosflm_data *mosflm; unsigned int opts; int status; @@ -531,6 +530,7 @@ void run_mosflm(struct image *image, UnitCell *cell) mosflm->pid = forkpty(&mosflm->pty, NULL, NULL, NULL); if ( mosflm->pid == -1 ) { ERROR("Failed to fork for MOSFLM\n"); + free(mosflm); return; } if ( mosflm->pid == 0 ) { @@ -590,12 +590,8 @@ void run_mosflm(struct image *image, UnitCell *cell) free(mosflm->rbuffer); waitpid(mosflm->pid, &status, 0); - /* Read the mosflm NEWMAT file and set cell candidate * - * Existence of this file means possible success. Pretty shady. */ - fail = read_newmat(mosflm->newmatfile, image); - if ( fail ) { - return; - } + /* Read the mosflm NEWMAT file and set cell candidate if found */ + read_newmat(mosflm->newmatfile, image); free(mosflm); } |