Hi Matthias
On Mon, Feb 18, 2019 at 12:56:57AM +0100, Matthias Berndt wrote:
I was recently trying to run connman on postmarketOS, and it crashed
while
setting up the routes for the wifi connection. I ran it again in gdb and
while I couldn't get a full stacktrace, I did see that it crashes because it
passes a NULL pointer to freeaddrinfo.
So I took a look at the source code and went through all the calls to that
function in connman. There aren't that many, and they all have error
handling code to prevent this - except for one. The attached patch should
fix it.
Great catch. Patch applied after adding a bit more explenation to the
commit message.
Thanks,
Daniel