All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-devel@nongnu.org
Subject: [PULL 02/20] contrib/elf2dmp: Remove unnecessary err flags
Date: Mon, 11 Mar 2024 19:12:23 +0000	[thread overview]
Message-ID: <20240311191241.4177990-3-peter.maydell@linaro.org> (raw)
In-Reply-To: <20240311191241.4177990-1-peter.maydell@linaro.org>

From: Akihiko Odaki <akihiko.odaki@daynix.com>

They are always evaluated to 1.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Tested-by: Viktor Prutyanov <viktor.prutyanov@phystech.edu>
Message-id: 20240307-elf2dmp-v4-1-4f324ad4d99d@daynix.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 contrib/elf2dmp/pdb.c | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/contrib/elf2dmp/pdb.c b/contrib/elf2dmp/pdb.c
index 40991f5f4c3..abf17c2e7c1 100644
--- a/contrib/elf2dmp/pdb.c
+++ b/contrib/elf2dmp/pdb.c
@@ -177,7 +177,6 @@ static int pdb_init_segments(struct pdb_reader *r)
 
 static int pdb_init_symbols(struct pdb_reader *r)
 {
-    int err = 0;
     PDB_SYMBOLS *symbols;
 
     symbols = pdb_ds_read_file(r, 3);
@@ -196,7 +195,6 @@ static int pdb_init_symbols(struct pdb_reader *r)
     /* Read global symbol table */
     r->modimage = pdb_ds_read_file(r, symbols->gsym_file);
     if (!r->modimage) {
-        err = 1;
         goto out_symbols;
     }
 
@@ -205,7 +203,7 @@ static int pdb_init_symbols(struct pdb_reader *r)
 out_symbols:
     g_free(symbols);
 
-    return err;
+    return 1;
 }
 
 static int pdb_reader_ds_init(struct pdb_reader *r, PDB_DS_HEADER *hdr)
@@ -228,7 +226,6 @@ static int pdb_reader_ds_init(struct pdb_reader *r, PDB_DS_HEADER *hdr)
 
 static int pdb_reader_init(struct pdb_reader *r, void *data)
 {
-    int err = 0;
     const char pdb7[] = "Microsoft C/C++ MSF 7.00";
 
     if (memcmp(data, pdb7, sizeof(pdb7) - 1)) {
@@ -241,17 +238,14 @@ static int pdb_reader_init(struct pdb_reader *r, void *data)
 
     r->ds.root = pdb_ds_read_file(r, 1);
     if (!r->ds.root) {
-        err = 1;
         goto out_ds;
     }
 
     if (pdb_init_symbols(r)) {
-        err = 1;
         goto out_root;
     }
 
     if (pdb_init_segments(r)) {
-        err = 1;
         goto out_sym;
     }
 
@@ -264,7 +258,7 @@ out_root:
 out_ds:
     pdb_reader_ds_exit(r);
 
-    return err;
+    return 1;
 }
 
 static void pdb_reader_exit(struct pdb_reader *r)
@@ -278,7 +272,6 @@ static void pdb_reader_exit(struct pdb_reader *r)
 int pdb_init_from_file(const char *name, struct pdb_reader *reader)
 {
     GError *gerr = NULL;
-    int err = 0;
     void *map;
 
     reader->gmf = g_mapped_file_new(name, TRUE, &gerr);
@@ -291,7 +284,6 @@ int pdb_init_from_file(const char *name, struct pdb_reader *reader)
     reader->file_size = g_mapped_file_get_length(reader->gmf);
     map = g_mapped_file_get_contents(reader->gmf);
     if (pdb_reader_init(reader, map)) {
-        err = 1;
         goto out_unmap;
     }
 
@@ -300,7 +292,7 @@ int pdb_init_from_file(const char *name, struct pdb_reader *reader)
 out_unmap:
     g_mapped_file_unref(reader->gmf);
 
-    return err;
+    return 1;
 }
 
 void pdb_exit(struct pdb_reader *reader)
-- 
2.34.1



  parent reply	other threads:[~2024-03-11 19:21 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-11 19:12 [PULL 00/20] target-arm queue Peter Maydell
2024-03-11 19:12 ` [PULL 01/20] hw/arm: Deprecate various old Arm machine types Peter Maydell
2024-03-11 19:12 ` Peter Maydell [this message]
2024-03-11 19:12 ` [PULL 03/20] contrib/elf2dmp: Assume error by default Peter Maydell
2024-03-11 19:12 ` [PULL 04/20] contrib/elf2dmp: Continue even contexts are lacking Peter Maydell
2024-03-11 19:12 ` [PULL 05/20] contrib/elf2dmp: Change pa_space_create() signature Peter Maydell
2024-03-11 19:12 ` [PULL 06/20] contrib/elf2dmp: Fix error reporting style in addrspace.c Peter Maydell
2024-03-11 19:12 ` [PULL 07/20] contrib/elf2dmp: Fix error reporting style in download.c Peter Maydell
2024-03-11 19:12 ` [PULL 08/20] contrib/elf2dmp: Fix error reporting style in pdb.c Peter Maydell
2024-03-11 19:12 ` [PULL 09/20] contrib/elf2dmp: Fix error reporting style in qemu_elf.c Peter Maydell
2024-03-11 19:12 ` [PULL 10/20] contrib/elf2dmp: Fix error reporting style in main.c Peter Maydell
2024-03-11 19:12 ` [PULL 11/20] contrib/elf2dmp: Always check for PA resolution failure Peter Maydell
2024-03-11 19:12 ` [PULL 12/20] contrib/elf2dmp: Always destroy PA space Peter Maydell
2024-03-11 19:12 ` [PULL 13/20] contrib/elf2dmp: Ensure segment fits in file Peter Maydell
2024-03-11 19:12 ` [PULL 14/20] contrib/elf2dmp: Use lduw_le_p() to read PDB Peter Maydell
2024-03-11 19:12 ` [PULL 15/20] contrib/elf2dmp: Use rol64() to decode Peter Maydell
2024-03-11 19:12 ` [PULL 16/20] MAINTAINERS: Add Akihiko Odaki as a elf2dmp reviewer Peter Maydell
2024-03-11 19:12 ` [PULL 17/20] contrib/elf2dmp: Use GPtrArray Peter Maydell
2024-03-11 19:12 ` [PULL 18/20] contrib/elf2dmp: Clamp QEMU note to file size Peter Maydell
2024-03-11 19:12 ` [PULL 19/20] contrib/elf2dmp: Ensure phdrs fit in file Peter Maydell
2024-03-11 19:12 ` [PULL 20/20] docs: update copyright date to the year 2024 Peter Maydell
2024-03-12 13:07 ` [PULL 00/20] target-arm queue Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240311191241.4177990-3-peter.maydell@linaro.org \
    --to=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.