All of lore.kernel.org
 help / color / mirror / Atom feed
From: Akihiko Odaki <akihiko.odaki@daynix.com>
To: Viktor Prutyanov <viktor.prutyanov@phystech.edu>,
	 Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-devel@nongnu.org, Akihiko Odaki <akihiko.odaki@daynix.com>
Subject: [PATCH v3 01/19] contrib/elf2dmp: Remove unnecessary err flags
Date: Wed, 06 Mar 2024 15:12:07 +0900	[thread overview]
Message-ID: <20240306-elf2dmp-v3-1-d74e6c3da49c@daynix.com> (raw)
In-Reply-To: <20240306-elf2dmp-v3-0-d74e6c3da49c@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>
---
 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 40991f5f4c34..abf17c2e7c12 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.44.0



  reply	other threads:[~2024-03-06  6:13 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-06  6:12 [PATCH v3 00/19] contrib/elf2dmp: Improve robustness Akihiko Odaki
2024-03-06  6:12 ` Akihiko Odaki [this message]
2024-03-06  6:12 ` [PATCH v3 02/19] contrib/elf2dmp: Assume error by default Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 03/19] contrib/elf2dmp: Continue even contexts are lacking Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 04/19] contrib/elf2dmp: Change pa_space_create() signature Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 05/19] contrib/elf2dmp: Fix error reporting style in addrspace.c Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 06/19] contrib/elf2dmp: Fix error reporting style in download.c Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 07/19] contrib/elf2dmp: Fix error reporting style in pdb.c Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 08/19] contrib/elf2dmp: Fix error reporting style in qemu_elf.c Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 09/19] contrib/elf2dmp: Fix error reporting style in main.c Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 10/19] contrib/elf2dmp: Always check for PA resolution failure Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 11/19] contrib/elf2dmp: Always destroy PA space Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 12/19] contrib/elf2dmp: Ensure segment fits in file Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 13/19] contrib/elf2dmp: Use lduw_le_p() to read PDB Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 14/19] contrib/elf2dmp: Use rol64() to decode Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 15/19] MAINTAINERS: Add Akihiko Odaki as a elf2dmp reviewer Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 16/19] contrib/elf2dmp: Build only for little endian host Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 17/19] contrib/elf2dmp: Use GPtrArray Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 18/19] contrib/elf2dmp: Clamp QEMU note to file size Akihiko Odaki
2024-03-06  6:12 ` [PATCH v3 19/19] contrib/elf2dmp: Ensure phdrs fit in file Akihiko Odaki

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=20240306-elf2dmp-v3-1-d74e6c3da49c@daynix.com \
    --to=akihiko.odaki@daynix.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=viktor.prutyanov@phystech.edu \
    /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.