{"id":419,"date":"2016-08-08T15:51:45","date_gmt":"2016-08-08T13:51:45","guid":{"rendered":"http:\/\/blogs.gentoo.org\/ago\/?p=419"},"modified":"2016-09-07T14:35:40","modified_gmt":"2016-09-07T12:35:40","slug":"wirouterkeyrec-signed-integer-overflow-in-agpf_get_serial-agpf-c","status":"publish","type":"post","link":"https:\/\/blogs.gentoo.org\/ago\/2016\/08\/08\/wirouterkeyrec-signed-integer-overflow-in-agpf_get_serial-agpf-c\/","title":{"rendered":"WiRouterKeyRec: signed integer overflow 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 shows the presence of a signed integer overflow in agpf_check_agpf.<\/p>\n<p>The complete UBSan 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\nsrc\/agpf.c:445:17: runtime error: signed integer overflow: 48230959 - -2101480424 cannot be represented in type 'int'\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-08: bug discovered<br \/>\n2016-08-08: bug reported to upstream<br \/>\n2016-08-08: 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=\"JsL3r2vOIU\" class=\"wp-embedded-content\"><p><a href=\"http:\/\/blogs.gentoo.org\/ago\/2016\/08\/08\/wirouterkeyrec-signed-integer-overflow-in-agpf_get_serial-agpf-c\/\">WiRouterKeyRec: signed integer overflow 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\/08\/wirouterkeyrec-signed-integer-overflow-in-agpf_get_serial-agpf-c\/embed\/#?secret=JsL3r2vOIU\" data-secret=\"JsL3r2vOIU\" width=\"600\" height=\"338\" title=\"&#8220;WiRouterKeyRec: signed integer overflow 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 shows the presence of a signed integer overflow in agpf_check_agpf. The complete UBSan output: # WiRouterKeyRec &#8211;config crash.agpf -s Alice-48230959 WiRouter KeyRec 1.1.2 &#8211; (C) 2011 Salvatore &hellip; <a href=\"https:\/\/blogs.gentoo.org\/ago\/2016\/08\/08\/wirouterkeyrec-signed-integer-overflow-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-6L","_links":{"self":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/419"}],"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=419"}],"version-history":[{"count":2,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/419\/revisions"}],"predecessor-version":[{"id":422,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/419\/revisions\/422"}],"wp:attachment":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/media?parent=419"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/categories?post=419"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/tags?post=419"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}