* [PATCH] ELF uapi: add spaces before '{'
@ 2022-10-04 10:25 Rolf Eike Beer
2022-10-06 3:33 ` Kees Cook
2022-10-18 7:22 ` Kees Cook
0 siblings, 2 replies; 3+ messages in thread
From: Rolf Eike Beer @ 2022-10-04 10:25 UTC (permalink / raw)
To: Eric Biederman, Kees Cook; +Cc: linux-mm, linux-kernel
When searching for a struct definition I often enough end up simply doing
git grep 'struct foobar {'
Sadly some of the ELF structs did not follow the usual coding style so they
were invisible.
Signed-off-by: Rolf Eike Beer <eb@emlix.com>
---
include/uapi/linux/elf.h | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/uapi/linux/elf.h b/include/uapi/linux/elf.h
index c7b056af9ef0..775a17b5b8ac 100644
--- a/include/uapi/linux/elf.h
+++ b/include/uapi/linux/elf.h
@@ -140,9 +140,9 @@ typedef __s64 Elf64_Sxword;
#define ELF64_ST_BIND(x) ELF_ST_BIND(x)
#define ELF64_ST_TYPE(x) ELF_ST_TYPE(x)
-typedef struct dynamic{
+typedef struct dynamic {
Elf32_Sword d_tag;
- union{
+ union {
Elf32_Sword d_val;
Elf32_Addr d_ptr;
} d_un;
@@ -173,7 +173,7 @@ typedef struct elf64_rel {
Elf64_Xword r_info; /* index and type of relocation */
} Elf64_Rel;
-typedef struct elf32_rela{
+typedef struct elf32_rela {
Elf32_Addr r_offset;
Elf32_Word r_info;
Elf32_Sword r_addend;
@@ -185,7 +185,7 @@ typedef struct elf64_rela {
Elf64_Sxword r_addend; /* Constant addend used to compute value */
} Elf64_Rela;
-typedef struct elf32_sym{
+typedef struct elf32_sym {
Elf32_Word st_name;
Elf32_Addr st_value;
Elf32_Word st_size;
@@ -206,7 +206,7 @@ typedef struct elf64_sym {
#define EI_NIDENT 16
-typedef struct elf32_hdr{
+typedef struct elf32_hdr {
unsigned char e_ident[EI_NIDENT];
Elf32_Half e_type;
Elf32_Half e_machine;
@@ -246,7 +246,7 @@ typedef struct elf64_hdr {
#define PF_W 0x2
#define PF_X 0x1
-typedef struct elf32_phdr{
+typedef struct elf32_phdr {
Elf32_Word p_type;
Elf32_Off p_offset;
Elf32_Addr p_vaddr;
--
2.37.3
--
Rolf Eike Beer, emlix GmbH, https://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055
emlix - smart embedded open source
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ELF uapi: add spaces before '{'
2022-10-04 10:25 [PATCH] ELF uapi: add spaces before '{' Rolf Eike Beer
@ 2022-10-06 3:33 ` Kees Cook
2022-10-18 7:22 ` Kees Cook
1 sibling, 0 replies; 3+ messages in thread
From: Kees Cook @ 2022-10-06 3:33 UTC (permalink / raw)
To: Rolf Eike Beer; +Cc: Eric Biederman, linux-mm, linux-kernel
On Tue, Oct 04, 2022 at 12:25:40PM +0200, Rolf Eike Beer wrote:
> When searching for a struct definition I often enough end up simply doing
>
> git grep 'struct foobar {'
>
> Sadly some of the ELF structs did not follow the usual coding style so they
> were invisible.
Thanks! Yes, I do the same. :)
I'll get this into -next after the merge window closes.
-Kees
--
Kees Cook
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ELF uapi: add spaces before '{'
2022-10-04 10:25 [PATCH] ELF uapi: add spaces before '{' Rolf Eike Beer
2022-10-06 3:33 ` Kees Cook
@ 2022-10-18 7:22 ` Kees Cook
1 sibling, 0 replies; 3+ messages in thread
From: Kees Cook @ 2022-10-18 7:22 UTC (permalink / raw)
To: eb, ebiederm; +Cc: Kees Cook, linux-kernel, linux-mm
On Tue, 04 Oct 2022 12:25:40 +0200, Rolf Eike Beer wrote:
> When searching for a struct definition I often enough end up simply doing
>
> git grep 'struct foobar {'
>
> Sadly some of the ELF structs did not follow the usual coding style so they
> were invisible.
>
> [...]
Applied to for-next/execve, thanks!
[1/1] ELF uapi: add spaces before '{'
https://git.kernel.org/kees/c/5ce85ed9d20e
--
Kees Cook
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-10-18 7:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-04 10:25 [PATCH] ELF uapi: add spaces before '{' Rolf Eike Beer
2022-10-06 3:33 ` Kees Cook
2022-10-18 7:22 ` Kees Cook
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).