David Kahurani : xmllite/reader: Avoid memory leak on error path.
Alexandre Julliard
julliard at winehq.org
Mon Jun 20 16:57:22 CDT 2022
Module: wine
Branch: master
Commit: 630c38b261960a814bdbdfdc8ab9c52bb4ef0179
URL: https://source.winehq.org/git/wine.git/?a=commit;h=630c38b261960a814bdbdfdc8ab9c52bb4ef0179
Author: David Kahurani <k.kahurani at gmail.com>
Date: Mon Jun 20 12:01:19 2022 +0300
xmllite/reader: Avoid memory leak on error path.
attr->value should not be freed as it was not allocated attr->localname
should instead having just being previously allocated and not freed
anywhere else
Signed-off-by: David Kahurani <k.kahurani at gmail.com>
---
dlls/xmllite/reader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dlls/xmllite/reader.c b/dlls/xmllite/reader.c
index 4045b4dfcca..d99f121e0cb 100644
--- a/dlls/xmllite/reader.c
+++ b/dlls/xmllite/reader.c
@@ -424,7 +424,7 @@ static HRESULT reader_add_attr(xmlreader *reader, strval *prefix, strval *localn
{
hr = reader_strvaldup(reader, value, &attr->value);
if (hr != S_OK)
- reader_free_strvalued(reader, &attr->value);
+ reader_free_strvalued(reader, &attr->localname);
}
if (hr != S_OK)
{
More information about the wine-cvs
mailing list