Nobody interested ? I guess my patch must be fine then, shall I commit ?<br><br>Joris,<br><br><div><span class="gmail_quote">On 10/20/07, <b class="gmail_sendername">Joris Guisson</b> <<a href="mailto:joris.guisson@gmail.com">

joris.guisson@gmail.com</a>> wrote:<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hi, I have noticed that there is a crash when you use KConfigDialog. When you add a page, then remove it again, and close the dialog with cancel (if your press OK, it doesn't crash).

<br><br>After some digging around in the code, I noticed there is this QMap managerForPage in KConfigDialog. Which gets iterated over when you press cancel. This map contains pointers to KConfigDialogManager objects (which are created per page), and those should be removed when a page is removed from the dialog, but this doesn't happen. 

KConfigDialogManager goes around checking a bunch of widgets which are part of a page and any properly coded program will delete those when it removes a page from a KConfigDialog. So in order to fix this crash you need clean up these KConfigDialogManager objects when the corresponding page is removed.

The attached patch does this. Joris, Btw, if you want to reproduce the crash, compile and install ktorrent (trunk/extagear/network/ktorrent), load the webinterface plugin in the settings dialog, then unload it again, and close the dialog with cancel.

<br><br><br>

<br clear="all"></blockquote></div><br>