{"action":"created","issue":{"url":"https://api.github.com/repos/bitcoin/bitcoin/issues/15043","repository_url":"https://api.github.com/repos/bitcoin/bitcoin","labels_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/15043/labels{/name}","comments_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/15043/comments","events_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/15043/events","html_url":"https://github.com/bitcoin/bitcoin/pull/15043","id":394443727,"node_id":"MDExOlB1bGxSZXF1ZXN0MjQxMTkwMjA5","number":15043,"title":"test: Build fuzz targets into seperate executables","user":{"login":"MarcoFalke","id":6399679,"node_id":"MDQ6VXNlcjYzOTk2Nzk=","avatar_url":"https://avatars0.githubusercontent.com/u/6399679?v=4","gravatar_id":"","url":"https://api.github.com/users/MarcoFalke","html_url":"https://github.com/MarcoFalke","followers_url":"https://api.github.com/users/MarcoFalke/followers","following_url":"https://api.github.com/users/MarcoFalke/following{/other_user}","gists_url":"https://api.github.com/users/MarcoFalke/gists{/gist_id}","starred_url":"https://api.github.com/users/MarcoFalke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarcoFalke/subscriptions","organizations_url":"https://api.github.com/users/MarcoFalke/orgs","repos_url":"https://api.github.com/users/MarcoFalke/repos","events_url":"https://api.github.com/users/MarcoFalke/events{/privacy}","received_events_url":"https://api.github.com/users/MarcoFalke/received_events","type":"User","site_admin":false},"labels":[{"id":61889416,"node_id":"MDU6TGFiZWw2MTg4OTQxNg==","url":"https://api.github.com/repos/bitcoin/bitcoin/labels/Build%20system","name":"Build system","color":"5319e7","default":false},{"id":997854106,"node_id":"MDU6TGFiZWw5OTc4NTQxMDY=","url":"https://api.github.com/repos/bitcoin/bitcoin/labels/Needs%20gitian%20build","name":"Needs gitian build","color":"8c9e07","default":false},{"id":62963516,"node_id":"MDU6TGFiZWw2Mjk2MzUxNg==","url":"https://api.github.com/repos/bitcoin/bitcoin/labels/Tests","name":"Tests","color":"d4c5f9","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-12-27T16:59:46Z","updated_at":"2019-01-02T11:53:32Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/15043","html_url":"https://github.com/bitcoin/bitcoin/pull/15043","diff_url":"https://github.com/bitcoin/bitcoin/pull/15043.diff","patch_url":"https://github.com/bitcoin/bitcoin/pull/15043.patch"},"body":"Currently our fuzzer is a single binary that decides on the first few bits of the buffer what target to pick. This is ineffective as the fuzzer needs to \"learn\" how the fuzz targets are organized and could get easily confused. Not to mention that the (seed) corpus can not be categorized by target, since targets might \"leak\" into each other. Also the corpus would potentially become invalid if we ever wanted to remove a target...\r\n\r\nSolve that by building each fuzz target into their own executable."},"comment":{"url":"https://api.github.com/repos/bitcoin/bitcoin/issues/comments/450845680","html_url":"https://github.com/bitcoin/bitcoin/pull/15043#issuecomment-450845680","issue_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/15043","id":450845680,"node_id":"MDEyOklzc3VlQ29tbWVudDQ1MDg0NTY4MA==","user":{"login":"laanwj","id":126646,"node_id":"MDQ6VXNlcjEyNjY0Ng==","avatar_url":"https://avatars0.githubusercontent.com/u/126646?v=4","gravatar_id":"","url":"https://api.github.com/users/laanwj","html_url":"https://github.com/laanwj","followers_url":"https://api.github.com/users/laanwj/followers","following_url":"https://api.github.com/users/laanwj/following{/other_user}","gists_url":"https://api.github.com/users/laanwj/gists{/gist_id}","starred_url":"https://api.github.com/users/laanwj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/laanwj/subscriptions","organizations_url":"https://api.github.com/users/laanwj/orgs","repos_url":"https://api.github.com/users/laanwj/repos","events_url":"https://api.github.com/users/laanwj/events{/privacy}","received_events_url":"https://api.github.com/users/laanwj/received_events","type":"User","site_admin":false},"created_at":"2019-01-02T11:53:32Z","updated_at":"2019-01-02T11:53:32Z","author_association":"MEMBER","body":"Concept ACK\r\n\r\nI think this should be behind a `configure` flag, building so many executables is going to be slow when statically linking, or with slow filesystems, and it contributes nothing to testing unless the user is planning to do fuzzing."}}
{"actor":{"display_login":"laanwj"}}