On Wed, 2016-08-10 at 21:31 +0530, Arun Raghavan wrote: > Using pa_xfree() irrespective of whether old_value is NULL or not to > avoid potentially confusing nest of if statements. > > CID: 1352052 > --- >  src/pulsecore/sink-input.c    | 4 +++- >  src/pulsecore/source-output.c | 4 +++- >  2 files changed, 6 insertions(+), 2 deletions(-) Looks good to me. -- Tanu