Blame SOURCES/0001-Fix-a-memory-leak-on-the-error-path-in-XpGetLocaleNe.patch
|
 |
9d96eb |
From a1e1a7b0b9a7bcededfc9166df58541675615699 Mon Sep 17 00:00:00 2001
|
|
|
9d96eb |
From: Adam Jackson <ajax@redhat.com>
|
|
|
9d96eb |
Date: Tue, 16 Oct 2018 12:39:26 -0400
|
|
|
9d96eb |
Subject: [PATCH] Fix a memory leak on the error path in XpGetLocaleNetString
|
|
|
9d96eb |
|
|
|
9d96eb |
After Data() we own the pointer again, so we can just free it before
|
|
|
9d96eb |
getting the reply.
|
|
|
9d96eb |
|
|
|
9d96eb |
Signed-off-by: Adam Jackson <ajax@redhat.com>
|
|
|
9d96eb |
---
|
|
|
9d96eb |
src/XpPrinter.c | 4 ++--
|
|
|
9d96eb |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
|
9d96eb |
|
|
|
9d96eb |
diff --git a/src/XpPrinter.c b/src/XpPrinter.c
|
|
|
9d96eb |
index 03b18c4..4e8cdc9 100644
|
|
|
9d96eb |
--- a/src/XpPrinter.c
|
|
|
9d96eb |
+++ b/src/XpPrinter.c
|
|
|
9d96eb |
@@ -118,14 +118,14 @@ XpGetPrinterList (
|
|
|
9d96eb |
if (req->localeLen)
|
|
|
9d96eb |
Data( dpy, (char *) locale, (long) req->localeLen );
|
|
|
9d96eb |
|
|
|
9d96eb |
+ XFree(locale);
|
|
|
9d96eb |
+
|
|
|
9d96eb |
if (! _XReply (dpy, (xReply *) &rep, 0, xFalse)) {
|
|
|
9d96eb |
UnlockDisplay(dpy);
|
|
|
9d96eb |
SyncHandle();
|
|
|
9d96eb |
return ( (XPPrinterList) NULL ); /* error */
|
|
|
9d96eb |
}
|
|
|
9d96eb |
|
|
|
9d96eb |
- XFree(locale);
|
|
|
9d96eb |
-
|
|
|
9d96eb |
*list_count = rep.listCount;
|
|
|
9d96eb |
|
|
|
9d96eb |
if (*list_count) {
|
|
|
9d96eb |
--
|
|
|
9d96eb |
2.19.0
|
|
|
9d96eb |
|