diff options
| author | Lennart Poettering <mznyfn@0pointer.de> | 2009-08-04 23:55:28 +0200 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2009-08-04 23:58:31 +0200 |
| commit | 50562a9d5d54270ab67f183013204d24cdbeff21 (patch) | |
| tree | 1840554a7e610d00b3c7ab6fc4d4aeeba1d7a6e1 /pulse/ctl_pulse.c | |
| parent | 59ccd2aca941b4653803f5415b4985ab35de0c35 (diff) | |
pulse: rework object destruction paths a bit
Make sure we deal better with partially initialized structs.
Don't check for pointer state before calling free() since free() does
that anyway.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'pulse/ctl_pulse.c')
| -rw-r--r-- | pulse/ctl_pulse.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/pulse/ctl_pulse.c b/pulse/ctl_pulse.c index 879f260..85863ed 100644 --- a/pulse/ctl_pulse.c +++ b/pulse/ctl_pulse.c @@ -606,11 +606,8 @@ static void pulse_close(snd_ctl_ext_t * ext) if (ctl->p) pulse_free(ctl->p); - if (ctl->source) - free(ctl->source); - if (ctl->sink) - free(ctl->sink); - + free(ctl->source); + free(ctl->sink); free(ctl); } |