Avoid calling emitResult() in failed case. internalJobDone() is called anyway, and...
authorMantia Andras <andris@stein.andtib>
Tue, 18 May 2010 17:45:04 +0000 (20:45 +0300)
committerMantia Andras <andris@stein.andtib>
Tue, 18 May 2010 17:45:04 +0000 (20:45 +0300)
commit9f29810f31e74fbcb9a6ae82adb27f5ad7768981
treecf35588fe3c83b7f6013a8bd119bd8238332d6cc
parent3b29c315dd38c41b507a084b779167f5032885fd
Avoid calling emitResult() in failed case. internalJobDone() is called anyway, and that calls emitResult(). The problem is: when the error message is shown, the eventLoop is reentered, the job is deleted. The emitResult() call after tries to use the deleted job -> crash. This might be only a workaround, but fixes bug 229557 (and its duplicates), bug 227823, bug 232444, 227584.
plugins/subversion/svnjobbase.cpp