From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============0800605746860070625=="
MIME-Version: 1.0
From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: elfutils-devel@lists.fedorahosted.org
Subject: Re: [PATCH 1/2] Add is_executable to Dwfl_Module.
Date: Thu, 11 Sep 2014 21:00:49 +0200
Message-ID: <20140911190049.GA1901@host2.jankratochvil.net>
In-Reply-To: 1410380142.27502.36.camel@bordewijk.wildebeest.org

--===============0800605746860070625==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

On Wed, 10 Sep 2014 22:15:42 +0200, Mark Wielaard wrote:
> So I propose a cleanup like the attached first.

While I find that as an improvement in general IMO on top of your patch the
changes could be done a bit differently.

Patricularly I at least miss there that 'e_ident'.


Jan

diff --git a/libdwfl/dwfl_segment_report_module.c b/libdwfl/dwfl_segment_re=
port_module.c
index 572f15b..be6950d 100644
--- a/libdwfl/dwfl_segment_report_module.c
+++ b/libdwfl/dwfl_segment_report_module.c
@@ -163,11 +163,11 @@ dwfl_segment_report_module (Dwfl *dwfl, int ndx, cons=
t char *name,
   /* Extract the information we need from the file header.  */
   unsigned char ei_class;
   unsigned char ei_data;
-  uint16_t e_type;
   union
   {
     Elf32_Ehdr e32;
     Elf64_Ehdr e64;
+    Elf_Ehdr e;
   } ehdr;
   GElf_Off phoff;
   uint_fast16_t phnum;
@@ -186,15 +186,14 @@ dwfl_segment_report_module (Dwfl *dwfl, int ndx, cons=
t char *name,
       .d_size =3D sizeof ehdr,
       .d_version =3D EV_CURRENT,
     };
-  ei_class =3D ((const unsigned char *) buffer)[EI_CLASS];
-  ei_data =3D ((const unsigned char *) buffer)[EI_DATA];
+  ei_class =3D ((const Elf_Ehdr *) buffer)->e_ident[EI_CLASS];
+  ei_data =3D ((const Elf_Ehdr *) buffer)->e_ident[EI_DATA];
   switch (ei_class)
     {
     case ELFCLASS32:
       xlatefrom.d_size =3D sizeof (Elf32_Ehdr);
       if (elf32_xlatetom (&xlateto, &xlatefrom, ei_data) =3D=3D NULL)
 	return finish ();
-      e_type =3D ehdr.e32.e_type;
       phoff =3D ehdr.e32.e_phoff;
       phnum =3D ehdr.e32.e_phnum;
       phentsize =3D ehdr.e32.e_phentsize;
@@ -207,7 +206,6 @@ dwfl_segment_report_module (Dwfl *dwfl, int ndx, const =
char *name,
       xlatefrom.d_size =3D sizeof (Elf64_Ehdr);
       if (elf64_xlatetom (&xlateto, &xlatefrom, ei_data) =3D=3D NULL)
 	return finish ();
-      e_type =3D ehdr.e64.e_type;
       phoff =3D ehdr.e64.e_phoff;
       phnum =3D ehdr.e64.e_phnum;
       phentsize =3D ehdr.e64.e_phentsize;
@@ -609,7 +607,7 @@ dwfl_segment_report_module (Dwfl *dwfl, int ndx, const =
char *name,
 =

   /* We'll use the name passed in or a stupid default if not DT_SONAME.  */
   if (name =3D=3D NULL)
-    name =3D e_type =3D=3D ET_EXEC ? "[exe]" : execlike ? "[pie]" : "[dso]=
";
+    name =3D ehdr.e.e_type =3D=3D ET_EXEC ? "[exe]" : execlike ? "[pie]" :=
 "[dso]";
 =

   void *soname =3D NULL;
   size_t soname_size =3D 0;
diff --git a/libelf/elf.h b/libelf/elf.h
index 40e87b2..47e7bc7 100644
--- a/libelf/elf.h
+++ b/libelf/elf.h
@@ -67,6 +67,14 @@ typedef Elf64_Half Elf64_Versym;
 typedef struct
 {
   unsigned char	e_ident[EI_NIDENT];	/* Magic number and other info */
+  uint16_t	e_type;			/* Object file type */
+  uint16_t	e_machine;		/* Architecture */
+  uint32_t	e_version;		/* Object file version */
+} Elf_Ehdr;
+
+typedef struct
+{
+  unsigned char	e_ident[EI_NIDENT];	/* Magic number and other info */
   Elf32_Half	e_type;			/* Object file type */
   Elf32_Half	e_machine;		/* Architecture */
   Elf32_Word	e_version;		/* Object file version */

--===============0800605746860070625==--