-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128627/#review98201
-----------------------------------------------------------




src/widgets/kurlrequester.cpp (line 307)
<https://git.reviewboard.kde.org/r/128627/#comment66149>

    Here is setted sender, it cannot be nullptr.


- Anthony Fieroni


On Авг. 8, 2016, 10:38 преди обяд, Jaime Torres Amate wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128627/
> -----------------------------------------------------------
> 
> (Updated Авг. 8, 2016, 10:38 преди обяд)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> avoid connecting KUrl from null
> 
> 
> Diffs
> -----
> 
>   src/widgets/kurlrequester.cpp 77e7c1d 
> 
> Diff: https://git.reviewboard.kde.org/r/128627/diff/
> 
> 
> Testing
> -------
> 
> before the patch, there was 4 connect(null, .....), that crashed within:
> 
> #0  0x00007ffff441f975 in __GI_raise (sig=sig@entry=6) at 
> ../sysdeps/unix/sysv/linux/raise.c:54
> #1  0x00007ffff4420d8a in __GI_abort () at abort.c:78
> #2  0x00007ffff50ca6e5 in QMessageLogger::warning(char const*, ...) const 
> (context=..., message=...)
>     at global/qlogging.cpp:1648
> #3  0x00007ffff50ca6e5 in QMessageLogger::warning(char const*, ...) const 
> (this=this@entry=0x7fffffffc270, msg=msg@entry=0x7ffff5412a58 
> "QObject::connect: invalid null parameter") at global/qlogging.cpp:557
> #4  0x00007ffff52ec011 in QObjectPrivate::connectImpl(QObject const*, int, 
> QObject const*, void**, QtPrivate::QSlotObjectBase*, Qt::ConnectionType, int 
> const*, QMetaObject const*) (sender=sender@entry=0x0, signal_index=<optimized 
> out>, receiver=receiver@entry=0xb7c970, slot=slot@entry=0x7fffffffc420, 
> slotObj=slotObj@entry=0xb58500, type=Qt::AutoConnection, types=0x0, 
> senderMetaObject=0x7ffff6a81900 <QLineEdit::staticMetaObject>) at 
> kernel/qobject.cpp:4674
> #5  0x00007ffff52ec2d2 in QObject::connectImpl(QObject const*, void**, 
> QObject const*, void**, QtPrivate::QSlotObjectBase*, Qt::ConnectionType, int 
> const*, QMetaObject const*) (sender=sender@entry=0x0, 
> signal=signal@entry=0x7fffffffc410, receiver=receiver@entry=0xb7c970, 
> slot=slot@entry=0x7fffffffc420, slotObj=0xb58500, type=Qt::AutoConnection, 
> types=0x0, senderMetaObject=0x7ffff6a81900 <QLineEdit::staticMetaObject>)
>     at kernel/qobject.cpp:4658
> #6  0x00007ffff7ef2465 in KUrlRequester::KUrlRequesterPrivate::init() 
> (type=Qt::AutoConnection, slot=
>     (void (KUrlRequester::*)(KUrlRequester * const, const QString &)) 
> 0x7ffff7eee1c0 <KUrlRequester::textChanged(QString const&)>, 
> receiver=0xb7c970, signal=
>     (void (QLineEdit::*)(QLineEdit * const, const QString &)) 0x7ffff64d34b0 
> <QLineEdit::textChanged(QString const&)>, sender=0x0) at 
> /usr/include/qt5/QtCore/qobject.h:239
> 
> applying the patch, kate starts (there is no connect from null)
> 
> 
> Thanks,
> 
> Jaime Torres Amate
> 
>

Reply via email to