[PATCH] qcap: Fix a COM reference leak in the audiorecord test.
Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com> --- dlls/qcap/tests/audiorecord.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
On 04/22/2019 02:38 AM, Damjan Jovanovic wrote:
Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com> --- dlls/qcap/tests/audiorecord.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
Thanks for finding the leak. "filter" can never be NULL at that point, and we also want to test that we really are releasing the last reference. I've sent an updated patch that addresses these.
On Mon, Apr 22, 2019 at 7:51 PM Zebediah Figura <z.figura12(a)gmail.com> wrote:
On 04/22/2019 02:38 AM, Damjan Jovanovic wrote:
Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com> --- dlls/qcap/tests/audiorecord.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
Thanks for finding the leak. "filter" can never be NULL at that point, and we also want to test that we really are releasing the last reference. I've sent an updated patch that addresses these.
Pleasure. It can be NULL, if IMoniker_BindToObject() fails.
On 4/22/19 1:06 PM, Damjan Jovanovic wrote:
On Mon, Apr 22, 2019 at 7:51 PM Zebediah Figura <z.figura12(a)gmail.com <mailto:z.figura12(a)gmail.com>> wrote:
On 04/22/2019 02:38 AM, Damjan Jovanovic wrote: > Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com <mailto:damjan.jov(a)gmail.com>> > --- > dlls/qcap/tests/audiorecord.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > > >
Thanks for finding the leak. "filter" can never be NULL at that point, and we also want to test that we really are releasing the last reference. I've sent an updated patch that addresses these.
Pleasure. It can be NULL, if IMoniker_BindToObject() fails.
IMoniker::BindToObject() shouldn't fail; that's checked above.
participants (2)
-
Damjan Jovanovic -
Zebediah Figura