{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4015","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4015/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4015/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4015/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/4015","id":109331603,"number":4015,"title":"cortexm_common: check for possible stack overflow in hardfault handler","user":{"login":"daniel-k","id":224128,"avatar_url":"https://avatars.githubusercontent.com/u/224128?v=3","gravatar_id":"","url":"https://api.github.com/users/daniel-k","html_url":"https://github.com/daniel-k","followers_url":"https://api.github.com/users/daniel-k/followers","following_url":"https://api.github.com/users/daniel-k/following{/other_user}","gists_url":"https://api.github.com/users/daniel-k/gists{/gist_id}","starred_url":"https://api.github.com/users/daniel-k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniel-k/subscriptions","organizations_url":"https://api.github.com/users/daniel-k/orgs","repos_url":"https://api.github.com/users/daniel-k/repos","events_url":"https://api.github.com/users/daniel-k/events{/privacy}","received_events_url":"https://api.github.com/users/daniel-k/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/arm","name":"arm","color":"dddddd"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/enhancement","name":"enhancement","color":"f17070"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/NEEDS%20SQUASHING","name":"NEEDS SQUASHING","color":"009800"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Ready%20for%20CI%20build","name":"Ready for CI build","color":"bfdadc"}],"state":"open","locked":false,"assignee":{"login":"gebart","id":317506,"avatar_url":"https://avatars.githubusercontent.com/u/317506?v=3","gravatar_id":"","url":"https://api.github.com/users/gebart","html_url":"https://github.com/gebart","followers_url":"https://api.github.com/users/gebart/followers","following_url":"https://api.github.com/users/gebart/following{/other_user}","gists_url":"https://api.github.com/users/gebart/gists{/gist_id}","starred_url":"https://api.github.com/users/gebart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gebart/subscriptions","organizations_url":"https://api.github.com/users/gebart/orgs","repos_url":"https://api.github.com/users/gebart/repos","events_url":"https://api.github.com/users/gebart/events{/privacy}","received_events_url":"https://api.github.com/users/gebart/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2015-10-01T16:03:52Z","updated_at":"2015-10-12T20:09:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/4015","html_url":"https://github.com/RIOT-OS/RIOT/pull/4015","diff_url":"https://github.com/RIOT-OS/RIOT/pull/4015.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/4015.patch"},"body":"When the hardfault handler executes it may case an overflow into the preceeding section in memory (bss) caused by printf. The already existent measure only make sure not leave valid ram, not leaving the own stack.\r\n\r\nThis PR will output a warning if the hardfault handler may have corrupted memory and display the corrupt range. If you happen to see this on your board it should be an indication that the handler stack is too small and raise the awareness that debugging the current state may be pointless."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/147506039","html_url":"https://github.com/RIOT-OS/RIOT/pull/4015#issuecomment-147506039","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4015","id":147506039,"user":{"login":"gebart","id":317506,"avatar_url":"https://avatars.githubusercontent.com/u/317506?v=3","gravatar_id":"","url":"https://api.github.com/users/gebart","html_url":"https://github.com/gebart","followers_url":"https://api.github.com/users/gebart/followers","following_url":"https://api.github.com/users/gebart/following{/other_user}","gists_url":"https://api.github.com/users/gebart/gists{/gist_id}","starred_url":"https://api.github.com/users/gebart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gebart/subscriptions","organizations_url":"https://api.github.com/users/gebart/orgs","repos_url":"https://api.github.com/users/gebart/repos","events_url":"https://api.github.com/users/gebart/events{/privacy}","received_events_url":"https://api.github.com/users/gebart/received_events","type":"User","site_admin":false},"created_at":"2015-10-12T20:09:04Z","updated_at":"2015-10-12T20:09:04Z","body":"ACK from me, but I would like another opinion on the increased complexity of the hardfault handler. @OlegHahm, @kaspar030, @haukepetersen maybe?\r\n\r\nNote: This PR only affects the `-DDEVELHELP` hardfault handler"}}