search

murgo / IrssiNotifier

notifies android phones from irc private messages
Apache License 2.0
171 stars 63 forks source link

Reset child signal handler. #160

Closed zigdon closed 10 years ago

zigdon commented 11 years ago

Not 100% sure why, but not having the sig handler caused us to think openssl wasn't available.

murgo commented 11 years ago

Did this manifest as decryption error on the device end?

I've received couple of bug reports in the last couple of weeks with people complaining about decryption errors if they were not attached to their shells or whatever. Is this related, and what does the $SIG{CHLD}='DEFAULT' actually do? :)

zigdon commented 11 years ago

No, not related to actual encryption - it was just failing to load the script in irssi, since openssl version didn't end up setting $! to 0. Reading this was what pointed me to the problem: http://stackoverflow.com/a/6299831/4913

On Tue, Nov 26, 2013 at 5:10 PM, Lauri Härsilä notifications@github.comwrote:

Did this manifest as decryption error on the device end?

I've received couple of bug reports in the last couple of weeks with people complaining about decryption errors if they were not attached to their shells or whatever. Is this related, and what does the $SIG{CHLD}='DEFAULT' actually do? :)

— Reply to this email directly or view it on GitHubhttps://github.com/murgo/IrssiNotifier/pull/160#issuecomment-29352394 .

Dan Boger