Review Request: Do not crash avahi DNSSD service

Commit Hook null at kde.org
Thu Aug 23 02:30:31 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105613/#review17890
-----------------------------------------------------------


This review has been submitted with commit 2b8f691aa97854580a67579368029146473f9256 by Christoph Feck to branch KDE/4.9.

- Commit Hook


On July 19, 2012, 1:55 a.m., Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105613/
> -----------------------------------------------------------
> 
> (Updated July 19, 2012, 1:55 a.m.)
> 
> 
> Review request for kdelibs, KTorrent, Joris Guisson, and Thiago Macieira.
> 
> 
> Description
> -------
> 
> This is my attempt to fix the crash. As far as I followed the code, the crash is because of m_group == 0 in serverStateChanged() when the avahi server is initially "registering" or in "collision", and not "running" yet. When e.g. KTorrent starts, it is possible that the avahi server needs some time until it is fully setup. The m_group is initialized when the first tryApply() is called, which only happens when server is initially in "running" state.
> 
> Additionally, the m_running variable was never reset to false in stop(), so the (sync) publish() would never return on failure.
> 
> 
> This addresses bug 215288.
>     http://bugs.kde.org/show_bug.cgi?id=215288
> 
> 
> Diffs
> -----
> 
>   dnssd/avahi-publicservice.cpp 52bef22 
> 
> Diff: http://git.reviewboard.kde.org/r/105613/diff/
> 
> 
> Testing
> -------
> 
> I have no idea what avahi is or how it works, so I did not test it.
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120823/5e479e2f/attachment.htm>


More information about the kde-core-devel mailing list