diff options
author | aszlig <aszlig@nix.build> | 2022-09-18 13:20:20 +0200 |
---|---|---|
committer | aszlig <aszlig@nix.build> | 2022-09-18 13:20:20 +0200 |
commit | f7cacf7fd58f79ff3ddaaa6fff87425090167ebd (patch) | |
tree | e8660ca46402dd50f3166227e792c1599efc196c /pkgs/aszlig/radare2 | |
parent | d27c8bc427251882e1dc861d0499170965b6720d (diff) |
radare2: Add patch to default to AT&T ASM syntax
Some personal history on this: I started to get used to AT&T syntax because it's the default in GDB and used that ever since until I one day starting to do some more reverse engineering using radare, which defaults to Intel syntax. Ever since then probably my most used command in GDB was "set disassembly-flavor intel" (because I was to lazy to add it to the config) because I constantly got confused by the source/destination operand swaps. This even happened during live reverse engineering at rC3 where I was confused about some function logic only to find out that I was viewing in AT&T syntax. Fast-forward to today: I'm debugging some application using WINE and winedbg uses AT&T syntax, which I didn't like at first. After reflecting on this for a while, I thought it would probably be better to get used to AT&T syntax again and switch everything to use AT&T for the following reasons: * Operands are more natural to read, since most libraries/APIs in higher level languages do it like this (well, except memcpy, strcpy, etc... maybe I now get confused by libc functions...) * AT&T syntax feels less verbose, for example "mov ecx, dword [eax]" is just "movl (%eax), %ecx" This very commit makes sure that radare2 now defaults to AT&T syntax instead of eg. ensuring that GDB uses Intel syntax by default. Signed-off-by: aszlig <aszlig@nix.build>
Diffstat (limited to 'pkgs/aszlig/radare2')
-rw-r--r-- | pkgs/aszlig/radare2/att-syntax-by-default.patch | 7 | ||||
-rw-r--r-- | pkgs/aszlig/radare2/default.nix | 5 |
2 files changed, 12 insertions, 0 deletions
diff --git a/pkgs/aszlig/radare2/att-syntax-by-default.patch b/pkgs/aszlig/radare2/att-syntax-by-default.patch new file mode 100644 index 00000000..908723b0 --- /dev/null +++ b/pkgs/aszlig/radare2/att-syntax-by-default.patch @@ -0,0 +1,7 @@ +diff --git a/libr/core/cconfig.c b/libr/core/cconfig.c +index 53252df6a..da45bad50 100644 +--- a/libr/core/cconfig.c ++++ b/libr/core/cconfig.c +@@ -3620,1 +3620,1 @@ R_API int r_core_config_init(RCore *core) { +- n = NODECB ("asm.syntax", "intel", &cb_asmsyntax); ++ n = NODECB ("asm.syntax", "att", &cb_asmsyntax); diff --git a/pkgs/aszlig/radare2/default.nix b/pkgs/aszlig/radare2/default.nix new file mode 100644 index 00000000..f786656b --- /dev/null +++ b/pkgs/aszlig/radare2/default.nix @@ -0,0 +1,5 @@ +{ radare2 }: + +radare2.overrideAttrs (drv: { + patches = (drv.patches or []) ++ [ ./att-syntax-by-default.patch ]; +}) |