{"action":"created","issue":{"url":"https://api.github.com/repos/radare/radare2/issues/2712","repository_url":"https://api.github.com/repos/radare/radare2","labels_url":"https://api.github.com/repos/radare/radare2/issues/2712/labels{/name}","comments_url":"https://api.github.com/repos/radare/radare2/issues/2712/comments","events_url":"https://api.github.com/repos/radare/radare2/issues/2712/events","html_url":"https://github.com/radare/radare2/issues/2712","id":85147885,"number":2712,"title":"Visual disassembly moving byte/byte can be confusing","user":{"login":"Maijin","id":1408600,"avatar_url":"https://avatars.githubusercontent.com/u/1408600?v=3","gravatar_id":"","url":"https://api.github.com/users/Maijin","html_url":"https://github.com/Maijin","followers_url":"https://api.github.com/users/Maijin/followers","following_url":"https://api.github.com/users/Maijin/following{/other_user}","gists_url":"https://api.github.com/users/Maijin/gists{/gist_id}","starred_url":"https://api.github.com/users/Maijin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Maijin/subscriptions","organizations_url":"https://api.github.com/users/Maijin/orgs","repos_url":"https://api.github.com/users/Maijin/repos","events_url":"https://api.github.com/users/Maijin/events{/privacy}","received_events_url":"https://api.github.com/users/Maijin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/radare/radare2/labels/consoleui","name":"consoleui","color":"5319e7"},{"url":"https://api.github.com/repos/radare/radare2/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/radare/radare2/labels/release","name":"release","color":"009800"},{"url":"https://api.github.com/repos/radare/radare2/labels/wrkshp","name":"wrkshp","color":"00ff20"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/radare/radare2/milestones/4","html_url":"https://github.com/radare/radare2/milestones/1.x","labels_url":"https://api.github.com/repos/radare/radare2/milestones/4/labels","id":430858,"number":4,"title":"1.x","description":"","creator":{"login":"XVilka","id":203261,"avatar_url":"https://avatars.githubusercontent.com/u/203261?v=3","gravatar_id":"","url":"https://api.github.com/users/XVilka","html_url":"https://github.com/XVilka","followers_url":"https://api.github.com/users/XVilka/followers","following_url":"https://api.github.com/users/XVilka/following{/other_user}","gists_url":"https://api.github.com/users/XVilka/gists{/gist_id}","starred_url":"https://api.github.com/users/XVilka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/XVilka/subscriptions","organizations_url":"https://api.github.com/users/XVilka/orgs","repos_url":"https://api.github.com/users/XVilka/repos","events_url":"https://api.github.com/users/XVilka/events{/privacy}","received_events_url":"https://api.github.com/users/XVilka/received_events","type":"User","site_admin":false},"open_issues":220,"closed_issues":74,"state":"open","created_at":"2013-09-18T15:40:16Z","updated_at":"2016-09-21T09:46:49Z","due_on":null,"closed_at":null},"comments":18,"created_at":"2015-06-04T15:30:06Z","updated_at":"2016-09-21T09:47:14Z","closed_at":null,"body":"When you go into visual mode but the offset is not aligned, it's really confusing for end-users.\r\nThis has been notified during workshops by the people especially in the data section see:\r\n\r\nhttp://radare.tv/a/70\r\n\r\nIf aligned (`s str.ASCII` in CLI mode):\r\n\r\n       |   ; DATA XREF from 0x0041199f (fcn.00411900)                           \r\n       |   ;-- str.ASCII:                                                       \r\n       |   0x00416ffc     .string \"ASCII\" ; len=6          \r\n\r\nIf not aligned (`s str.ASCII+1` in CLI mode):\r\n\r\n       |   0x00416ffd    53             push rbx                                \r\n       |   0x00416ffe    434949004348   add byte [r11 + 0x48], al         \r\n\r\nIf it's not aligned it displays assembly code which is wrong. Same happen when you manipulate the code part: \r\n\r\nhttp://radare.tv/a/71\r\n\r\nAligned :\r\n\r\n    |          0x004028cc    53             push rbx                                \r\n    |          0x004028cd    89fb           mov ebx, edi  \r\n\r\nCan change to this if not aligned which is completly diferrent and prone to error: \r\n\r\n    |          0x004028ce    fb             sti                                     \r\n    |             read (0x0,0x0,0x0) ; section_end.phdr1+-8494577 \r\n\r\nThe fact you can move is very useful when you analyse undefined datas, and unknown stuff so it must be kept in a way, but it should be able to automatically allow the user to have the possibility to use kind of automatic alignement. (asm.midflags improvement?)"},"comment":{"url":"https://api.github.com/repos/radare/radare2/issues/comments/248563278","html_url":"https://github.com/radare/radare2/issues/2712#issuecomment-248563278","issue_url":"https://api.github.com/repos/radare/radare2/issues/2712","id":248563278,"user":{"login":"radare","id":917142,"avatar_url":"https://avatars.githubusercontent.com/u/917142?v=3","gravatar_id":"","url":"https://api.github.com/users/radare","html_url":"https://github.com/radare","followers_url":"https://api.github.com/users/radare/followers","following_url":"https://api.github.com/users/radare/following{/other_user}","gists_url":"https://api.github.com/users/radare/gists{/gist_id}","starred_url":"https://api.github.com/users/radare/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/radare/subscriptions","organizations_url":"https://api.github.com/users/radare/orgs","repos_url":"https://api.github.com/users/radare/repos","events_url":"https://api.github.com/users/radare/events{/privacy}","received_events_url":"https://api.github.com/users/radare/received_events","type":"User","site_admin":false},"created_at":"2016-09-21T09:47:14Z","updated_at":"2016-09-21T09:47:14Z","body":"removing @ret2libc as long as i cant expect this to be done for the next release or by him at all"}}
{"actor":{"display_login":"radare"}}