From f63926fbac83420856807838970b08b16c257944 Mon Sep 17 00:00:00 2001 From: Sundeep Anand Date: Mon, 10 May 2021 13:17:39 +0530 Subject: [PATCH] fix a few issues detected by coverity static analyzers * gettext-tools/src/x-csharp.c: free buffer in comment_line_end * gettext-tools/src/x-java.c: free buffer in comment_line_end * gettext-tools/src/x-rst.c: free s1 in extract_rsj * gettext-tools/src/xg-arglist-callshape.c: free xcomment in split_keywordspec --- gettext-tools/src/x-csharp.c | 1 + gettext-tools/src/x-java.c | 1 + gettext-tools/src/x-rst.c | 3 +++ gettext-tools/src/xg-arglist-callshape.c | 1 + 4 files changed, 6 insertions(+) diff --git a/gettext-tools/src/x-csharp.c b/gettext-tools/src/x-csharp.c index 13d4a5310..669db2fb8 100644 --- a/gettext-tools/src/x-csharp.c +++ b/gettext-tools/src/x-csharp.c @@ -561,6 +561,7 @@ comment_line_end (size_t chars_to_remove) buffer[buflen] = '\0'; savable_comment_add (buffer); lexical_context = lc_outside; + free (buffer); } diff --git a/gettext-tools/src/x-java.c b/gettext-tools/src/x-java.c index e89ab95f4..6c1b4a7fe 100644 --- a/gettext-tools/src/x-java.c +++ b/gettext-tools/src/x-java.c @@ -461,6 +461,7 @@ comment_line_end (size_t chars_to_remove) --buflen; buffer[buflen] = '\0'; savable_comment_add (buffer); + free (buffer); } diff --git a/gettext-tools/src/x-rst.c b/gettext-tools/src/x-rst.c index cde1ee6c0..82f645f80 100644 --- a/gettext-tools/src/x-rst.c +++ b/gettext-tools/src/x-rst.c @@ -651,6 +651,8 @@ extract_rsj (FILE *f, else goto invalid_rsj; + free (s1); + /* Parse a comma. */ c = phase2_getc (); if (c == '}') @@ -667,6 +669,7 @@ extract_rsj (FILE *f, fp = NULL; real_file_name = NULL; + free (logical_file_name); logical_file_name = NULL; line_number = 0; diff --git a/gettext-tools/src/xg-arglist-callshape.c b/gettext-tools/src/xg-arglist-callshape.c index 06b70760b..2189aac57 100644 --- a/gettext-tools/src/xg-arglist-callshape.c +++ b/gettext-tools/src/xg-arglist-callshape.c @@ -118,6 +118,7 @@ split_keywordspec (const char *spec, memcpy (xcomment, xcomment_start, xcomment_len); xcomment[xcomment_len] = '\0'; string_list_append (&xcomments, xcomment); + free (xcomment); } else break; -- 2.31.1