{"id":406,"date":"2016-08-05T16:04:30","date_gmt":"2016-08-05T14:04:30","guid":{"rendered":"http:\/\/blogs.gentoo.org\/ago\/?p=406"},"modified":"2016-09-07T14:35:53","modified_gmt":"2016-09-07T12:35:53","slug":"wirouterkeyrec-divide-by-zero-in-agpf_get_serial-agpf-c","status":"publish","type":"post","link":"https:\/\/blogs.gentoo.org\/ago\/2016\/08\/05\/wirouterkeyrec-divide-by-zero-in-agpf_get_serial-agpf-c\/","title":{"rendered":"WiRouterKeyRec: divide-by-zero in agpf_get_serial (agpf.c)"},"content":{"rendered":"<p><strong>Description<\/strong>:<br \/>\n<a href=\"http:\/\/www.salvatorefresta.net\/tools\/\">WiRouterKeyRec<\/a> is a recovery tool for wpa passphrase.<\/p>\n<p>A crafted AGPF config causes a divide-by-zero in agpf_get_serial.<\/p>\n<p>The complete ASan output:<\/p>\n<pre><font size=\"2\">WiRouterKeyRec --config crash.agpf -s Alice-48230959  \r\n\r\nWiRouter KeyRec 1.1.2 - (C) 2011 Salvatore Fresta\r\nhttp:\/\/www.salvatorefresta.net\r\n\r\nASAN:DEADLYSIGNAL\r\n=================================================================\r\n==27225==ERROR: AddressSanitizer: FPE on unknown address 0x0000005019fc (pc 0x0000005019fc bp 0x7fffe1f6fbe0 sp 0x7fffe1f6fa00 T0)\r\n    #0 0x5019fb in agpf_get_serial \/tmp\/portage\/app-crypt\/WiRouterKeyRec-1.1.2\/work\/WiRouter_KeyRec_1.1.2\/src\/agpf.c:445:20\r\n    #1 0x5019fb in agpf_get_config \/tmp\/portage\/app-crypt\/WiRouterKeyRec-1.1.2\/work\/WiRouter_KeyRec_1.1.2\/src\/agpf.c:355                                                                       \r\n    #2 0x4f510f in wr_get_keys \/tmp\/portage\/app-crypt\/WiRouterKeyRec-1.1.2\/work\/WiRouter_KeyRec_1.1.2\/src\/wirouterkeyrec.c:480:28                                                              \r\n    #3 0x4f2238 in main \/tmp\/portage\/app-crypt\/WiRouterKeyRec-1.1.2\/work\/WiRouter_KeyRec_1.1.2\/src\/wirouterkeyrec.c:307:18                                                                     \r\n    #4 0x7fdbc7f6161f in __libc_start_main \/var\/tmp\/portage\/sys-libs\/glibc-2.22-r4\/work\/glibc-2.22\/csu\/libc-start.c:289                                                                        \r\n    #5 0x418c28 in getenv (\/usr\/bin\/WiRouterKeyRec+0x418c28)                                                                                                                                   \r\n                                                                                                                                                                                               \r\nAddressSanitizer can not provide additional info.                                                                                                                                              \r\nSUMMARY: AddressSanitizer: FPE \/tmp\/portage\/app-crypt\/WiRouterKeyRec-1.1.2\/work\/WiRouter_KeyRec_1.1.2\/src\/agpf.c:445:20 in agpf_get_serial                                                     \r\n==27225==ABORTING\r\n<\/font><\/pre>\n<p><strong>Affected version:<\/strong><br \/>\n1.1.2<\/p>\n<p><strong>Fixed version:<\/strong><br \/>\nN\/A<\/p>\n<p><strong>Commit fix:<\/strong><br \/>\nN\/A<\/p>\n<p><strong>Credit:<\/strong><br \/>\nThis bug was discovered by Agostino Sarubbo of Gentoo.<\/p>\n<p><strong>Timeline:<\/strong><br \/>\n2016-08-04: bug discovered<br \/>\n2016-08-05: bug reported to upstream<br \/>\n2016-08-05: blog post about the issue<\/p>\n<p><strong>Note:<\/strong><br \/>\nThis bug was found with <a href=\"http:\/\/lcamtuf.coredump.cx\/afl\">American Fuzzy Lop<\/a>.<\/p>\n<p><strong>Permalink:<\/strong><\/p>\n<blockquote data-secret=\"czMLtRhBRU\" class=\"wp-embedded-content\"><p><a href=\"http:\/\/blogs.gentoo.org\/ago\/2016\/08\/05\/wirouterkeyrec-divide-by-zero-in-agpf_get_serial-agpf-c\/\">WiRouterKeyRec: divide-by-zero in agpf_get_serial (agpf.c)<\/a><\/p><\/blockquote>\n<p><iframe class=\"wp-embedded-content\" sandbox=\"allow-scripts\" security=\"restricted\" style=\"position: absolute; clip: rect(1px, 1px, 1px, 1px);\" src=\"http:\/\/blogs.gentoo.org\/ago\/2016\/08\/05\/wirouterkeyrec-divide-by-zero-in-agpf_get_serial-agpf-c\/embed\/#?secret=czMLtRhBRU\" data-secret=\"czMLtRhBRU\" width=\"600\" height=\"338\" title=\"&#8220;WiRouterKeyRec: divide-by-zero in agpf_get_serial (agpf.c)&#8221; &#8212; agostino&#039;s blog\" frameborder=\"0\" marginwidth=\"0\" marginheight=\"0\" scrolling=\"no\"><\/iframe><\/p>\n","protected":false},"excerpt":{"rendered":"<p>Description: WiRouterKeyRec is a recovery tool for wpa passphrase. A crafted AGPF config causes a divide-by-zero in agpf_get_serial. The complete ASan output: WiRouterKeyRec &#8211;config crash.agpf -s Alice-48230959 WiRouter KeyRec 1.1.2 &#8211; (C) 2011 Salvatore Fresta http:\/\/www.salvatorefresta.net ASAN:DEADLYSIGNAL ================================================================= ==27225==ERROR: AddressSanitizer: &hellip; <a href=\"https:\/\/blogs.gentoo.org\/ago\/2016\/08\/05\/wirouterkeyrec-divide-by-zero-in-agpf_get_serial-agpf-c\/\">Continue reading <span class=\"meta-nav\">&rarr;<\/span><\/a><\/p>\n","protected":false},"author":140,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"spay_email":"","jetpack_publicize_message":"","jetpack_is_tweetstorm":false,"jetpack_publicize_feature_enabled":true},"categories":[12,10],"tags":[],"jetpack_publicize_connections":[],"jetpack_featured_media_url":"","jetpack_sharing_enabled":true,"jetpack_shortlink":"https:\/\/wp.me\/p2EaBc-6y","_links":{"self":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/406"}],"collection":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/users\/140"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/comments?post=406"}],"version-history":[{"count":2,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/406\/revisions"}],"predecessor-version":[{"id":408,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/406\/revisions\/408"}],"wp:attachment":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/media?parent=406"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/categories?post=406"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/tags?post=406"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}