{"action":"created","issue":{"url":"https://api.github.com/repos/enzienaudio/heavy/issues/78","repository_url":"https://api.github.com/repos/enzienaudio/heavy","labels_url":"https://api.github.com/repos/enzienaudio/heavy/issues/78/labels{/name}","comments_url":"https://api.github.com/repos/enzienaudio/heavy/issues/78/comments","events_url":"https://api.github.com/repos/enzienaudio/heavy/issues/78/events","html_url":"https://github.com/enzienaudio/heavy/issues/78","id":140023879,"number":78,"title":"Pure data external: #HV_TO_PD sender hook crashes on messages with selector","user":{"login":"grrrr","id":163366,"avatar_url":"https://avatars.githubusercontent.com/u/163366?v=3","gravatar_id":"","url":"https://api.github.com/users/grrrr","html_url":"https://github.com/grrrr","followers_url":"https://api.github.com/users/grrrr/followers","following_url":"https://api.github.com/users/grrrr/following{/other_user}","gists_url":"https://api.github.com/users/grrrr/gists{/gist_id}","starred_url":"https://api.github.com/users/grrrr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grrrr/subscriptions","organizations_url":"https://api.github.com/users/grrrr/orgs","repos_url":"https://api.github.com/users/grrrr/repos","events_url":"https://api.github.com/users/grrrr/events{/privacy}","received_events_url":"https://api.github.com/users/grrrr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-03-10T22:08:57Z","updated_at":"2016-03-22T06:34:19Z","closed_at":null,"body":"Sending a message from a heavy patch out to a Pd host is ok for floats, but not for more complex messages involving a selector. The compiled external crashes with a bus error.\r\nFind below an example for a _main.pd where this happens. Float messages sent into the external are converted to messages with a selector - the external crashes immediately on sending the message.\r\nI am suspecting a memory allocation/overrun problem in the sendHook function in the *_tilde.c file.\r\n\r\n```\r\n#N canvas 75 215 264 148 10;\r\n#X obj 43 23 adc~;\r\n#X obj 42 71 dac~;\r\n#X obj 124 95 s #HV_TO_PD;\r\n#X obj 124 27 r in;\r\n#X msg 124 65 selector \\$1;\r\n#X connect 0 0 1 0;\r\n#X connect 0 1 1 1;\r\n#X connect 3 0 4 0;\r\n#X connect 4 0 2 0;\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/enzienaudio/heavy/issues/comments/199658951","html_url":"https://github.com/enzienaudio/heavy/issues/78#issuecomment-199658951","issue_url":"https://api.github.com/repos/enzienaudio/heavy/issues/78","id":199658951,"user":{"login":"mhroth","id":25988,"avatar_url":"https://avatars.githubusercontent.com/u/25988?v=3","gravatar_id":"","url":"https://api.github.com/users/mhroth","html_url":"https://github.com/mhroth","followers_url":"https://api.github.com/users/mhroth/followers","following_url":"https://api.github.com/users/mhroth/following{/other_user}","gists_url":"https://api.github.com/users/mhroth/gists{/gist_id}","starred_url":"https://api.github.com/users/mhroth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhroth/subscriptions","organizations_url":"https://api.github.com/users/mhroth/orgs","repos_url":"https://api.github.com/users/mhroth/repos","events_url":"https://api.github.com/users/mhroth/events{/privacy}","received_events_url":"https://api.github.com/users/mhroth/received_events","type":"User","site_admin":false},"created_at":"2016-03-22T06:34:19Z","updated_at":"2016-03-22T06:34:19Z","body":"@grrrr Thanks for the patch! I've finally had a chance to review it and have merged your patch and pushed the changes to the server. Makes sense!"}}