forked from jengelh/wayland
Jan Engelhardt
0c574da87f
OBS-URL: https://build.opensuse.org/package/show/X11:Wayland/wayland?expand=0&rev=7
60 lines
1.8 KiB
Diff
60 lines
1.8 KiB
Diff
From 1bade73b6b58cf2c3187b0fca72f4aec8301c4a3 Mon Sep 17 00:00:00 2001
|
|
From: =?UTF-8?q?Kristian=20H=C3=B8gsberg?= <krh@bitplanet.net>
|
|
Date: Sat, 20 Oct 2012 11:38:57 -0400
|
|
Subject: [PATCH] scanner: Fix valgrind errors
|
|
|
|
---
|
|
src/scanner.c | 15 +++++++--------
|
|
1 file changed, 7 insertions(+), 8 deletions(-)
|
|
|
|
diff --git a/src/scanner.c b/src/scanner.c
|
|
index d0aafec..4a4e605 100644
|
|
--- a/src/scanner.c
|
|
+++ b/src/scanner.c
|
|
@@ -393,6 +393,8 @@ start_element(void *data, const char *element_name, const char **atts)
|
|
case OBJECT:
|
|
if (interface_name)
|
|
arg->interface_name = strdup(interface_name);
|
|
+ else
|
|
+ arg->interface_name = NULL;
|
|
break;
|
|
default:
|
|
if (interface_name != NULL)
|
|
@@ -449,10 +451,7 @@ start_element(void *data, const char *element_name, const char **atts)
|
|
fail(ctx, "description without summary");
|
|
|
|
description = malloc(sizeof *description);
|
|
- if (summary)
|
|
- description->summary = strdup(summary);
|
|
- else
|
|
- description->summary = NULL;
|
|
+ description->summary = strdup(summary);
|
|
|
|
if (ctx->message)
|
|
ctx->message->description = description;
|
|
@@ -476,10 +475,9 @@ end_element(void *data, const XML_Char *name)
|
|
strndup(ctx->character_data,
|
|
ctx->character_data_length);
|
|
} else if (strcmp(name, "description") == 0) {
|
|
- char *text = strndup(ctx->character_data,
|
|
- ctx->character_data_length);
|
|
- if (text)
|
|
- ctx->description->text = text;
|
|
+ ctx->description->text =
|
|
+ strndup(ctx->character_data,
|
|
+ ctx->character_data_length);
|
|
ctx->description = NULL;
|
|
} else if (strcmp(name, "request") == 0 ||
|
|
strcmp(name, "event") == 0) {
|
|
@@ -1147,6 +1145,7 @@ int main(int argc, char *argv[])
|
|
protocol.type_index = 0;
|
|
protocol.null_run_length = 0;
|
|
protocol.copyright = NULL;
|
|
+ memset(&ctx, 0, sizeof ctx);
|
|
ctx.protocol = &protocol;
|
|
|
|
ctx.filename = "<stdin>";
|
|
--
|
|
1.7.10.4
|
|
|