Summary: | [knewstuff3] ASSERT: "rating <= 100" in file provider.cpp:601 | ||
---|---|---|---|
Product: | [Frameworks and Libraries] frameworks-attica | Reporter: | JR <zorael> |
Component: | general | Assignee: | Jeremy Whiting <jpwhiting> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | 507laird, andresbajotierra, Army_Brat_93, ashl1future, asraniel, astromme, bou.gui, bvgz, chazz5557, chesstercl, dont.eat.acid, francesco.marinucci, garystriplett, gladhorn, hara75gr, jazzemb, joelcacooney, johanp, juanfco.ruiz, kde, kitzune.1517, lime, linuxg33k4life, lobotexx, martin.lopusny, mossad, nck.s.hayes, nraymond, oldium.pro, ostre.gatto, riccardoescher, salcolon, seajey.serg, sebadamus, sergiusmx, sss123next, sudnikand, sven, tauraniaakant, tomfasching, tristanvillers, whetherharder, wuseldusel |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 4.7.1 | |
Sentry Crash Report: | |||
Attachments: |
New crash information added by DrKonqi
New crash information added by DrKonqi New crash information added by DrKonqi New crash information added by DrKonqi New crash information added by DrKonqi New crash information added by DrKonqi New crash information added by DrKonqi New crash information added by DrKonqi Patch to fix the voting problem New crash information added by DrKonqi |
Description
JR
2010-12-29 13:32:45 UTC
Created attachment 55393 [details]
New crash information added by DrKonqi
systemsettings (1.0) on KDE Platform 4.5.90 (4.6 RC1) using Qt 4.7.0
- What I was doing when the application crashed:
Browsing the KDM GHNS widget. It crashes 100% of the time if I click 'details' on the KStarboard KDM
I am also using Kubuntu 10.10 with the 4.4.90 packages.
-- Backtrace (Reduced):
#13 0x0783a645 in qt_assert (assertion=0x69ff1f "rating <= 100", file=0x69fbdc "/build/buildd/attica-0.2.0/lib/provider.cpp", line=601) at global/qglobal.cpp:2004
#14 0x00692908 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib/libattica.so.0
#15 0x01de1e98 in KNS3::AtticaProvider::vote (this=0x9613ad8, entry=..., rating=110) at ../../../knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#16 0x01dba6be in KNS3::Engine::vote (this=0x9639490, entry=..., rating=110) at ../../../knewstuff/knewstuff3/core/engine.cpp:481
#17 0x01dd3bd3 in KNS3::EntryDetails::ratingChanged (this=0x96689a0, rating=11) at ../../../knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
for this bug that i can't reproduce because i can't see anything with more than 100% of rating. What about displaying at 100% ? diff --git a/lib/provider.cpp b/lib/provider.cpp index d9298a7..51e5f41 100644 --- a/lib/provider.cpp +++ b/lib/provider.cpp @@ -1100,7 +1100,10 @@ PostJob* Provider::voteForContent(const QString& contentId, uint rating) } // according to OCS API, the rating is 0..100 - Q_ASSERT(rating <= 100); + if (rating > 100) { + rating = 100 ; + } + StringMap postParameters; postParameters.insert(QLatin1String( "vote" ), QString::number(rating)); qDebug() << "vote: " << QString::number(rating); commit fc00143ae0e57194eb2a3a58c87263d6969e36e3 branch master Author: Nicolas Lécureuil <neoclust.kde@free.fr> Date: Mon Jan 3 23:58:09 2011 +0100 Fix crash when the rating is superior to 100 BUG: 261530 diff --git a/lib/provider.cpp b/lib/provider.cpp index d9298a7..24994cf 100644 --- a/lib/provider.cpp +++ b/lib/provider.cpp @@ -1100,7 +1100,11 @@ PostJob* Provider::voteForContent(const QString& contentId, uint rating) } // according to OCS API, the rating is 0..100 - Q_ASSERT(rating <= 100); + if (rating > 100) { + qWarning() << "Rating cannot be superior to 100, fallback to 100." ; + rating = 100 ; + } + StringMap postParameters; postParameters.insert(QLatin1String( "vote" ), QString::number(rating)); qDebug() << "vote: " << QString::number(rating); *** Bug 262277 has been marked as a duplicate of this bug. *** Created attachment 55760 [details]
New crash information added by DrKonqi
systemsettings (1.0) on KDE Platform 4.5.95 (4.6 RC2) using Qt 4.7.0
- What I was doing when the application crashed:
I tried to install new themes and splash screens. System settings crashed 4 times.
-- Backtrace (Reduced):
#13 0x0408e645 in qt_assert (assertion=0x342f1f "rating <= 100", file=0x342bdc "/build/buildd/attica-0.2.0/lib/provider.cpp", line=601) at global/qglobal.cpp:2004
#14 0x00335908 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib/libattica.so.0
#15 0x02ea4e98 in KNS3::AtticaProvider::vote (this=0xaa6d230, entry=..., rating=110) at ../../../knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#16 0x02e7d6be in KNS3::Engine::vote (this=0xa144578, entry=..., rating=110) at ../../../knewstuff/knewstuff3/core/engine.cpp:481
#17 0x02e96bd3 in KNS3::EntryDetails::ratingChanged (this=0xaa8b2e0, rating=11) at ../../../knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
Created attachment 56454 [details]
New crash information added by DrKonqi
plasma-desktop (0.4) on KDE Platform 4.6.00 (4.6.0) using Qt 4.7.0
Same crash in KDE 4.6 released yesterday.
Shouldn't this fix be there?
-- Backtrace (Reduced):
#13 0xb645e645 in qt_assert (assertion=0xb373df1f "rating <= 100", file=0xb373dbdc "/build/buildd/attica-0.2.0/lib/provider.cpp", line=601) at global/qglobal.cpp:2004
#14 0xb3730908 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib/libattica.so.0
#15 0xb6e05eb8 in KNS3::AtticaProvider::vote (this=0xaa06570, entry=..., rating=120) at ../../../knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#16 0xb6dde6be in KNS3::Engine::vote (this=0xacc4a80, entry=..., rating=120) at ../../../knewstuff/knewstuff3/core/engine.cpp:481
#17 0xb6df7bf3 in KNS3::EntryDetails::ratingChanged (this=0xaecc8c8, rating=12) at ../../../knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
[Comment from a bug triager] The bug and the fix are in "libattica", may be your version of that library is not the latest. (or there wasn't a release including the fix) *** Bug 264511 has been marked as a duplicate of this bug. *** *** Bug 264790 has been marked as a duplicate of this bug. *** *** Bug 265458 has been marked as a duplicate of this bug. *** *** Bug 265788 has been marked as a duplicate of this bug. *** Created attachment 56996 [details]
New crash information added by DrKonqi
systemsettings (1.0) on KDE Platform 4.6.00 (4.6.0) "release 381" using Qt 4.7.1
- What I was doing when the application crashed:
just previewing some icons sets and suddenly the preview crashed
-- Backtrace (Reduced):
#11 0x00007f06ebbaec0a in qt_assert (assertion=0x2ae3 <Address 0x2ae3 out of bounds>, file=0x6 <Address 0x6 out of bounds>, line=-1) at global/qglobal.cpp:2027
#12 0x00007f06ea8fa288 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib64/libattica.so.0
#13 0x00007f06d9f58603 in KNS3::AtticaProvider::vote (this=0xe09a80, entry=<value optimized out>, rating=<value optimized out>)
at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#14 0x00007f06d9f6673d in KNS3::Engine::vote (this=<value optimized out>, entry=..., rating=110) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/core/engine.cpp:481
#15 0x00007f06d9f667d2 in KNS3::EntryDetails::ratingChanged (this=0xb905d0, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
*** Bug 266019 has been marked as a duplicate of this bug. *** *** Bug 266348 has been marked as a duplicate of this bug. *** Created attachment 57484 [details]
New crash information added by DrKonqi
systemsettings (1.0) on KDE Platform 4.6.00 (4.6.0) using Qt 4.7.0
- What I was doing when the application crashed: I was looking through the different themes, i clicked to preview the slimglow, then it crashed. I had just enabled desktop effects which had crashed a little while earlier. It seems like there is a memory leak somewhere.
-- Backtrace (Reduced):
#11 0x00007f78ac4dc0a8 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib/libattica.so.0
#12 0x00007f7895284c20 in KNS3::AtticaProvider::vote (this=0x202a780, entry=<value optimized out>, rating=110) at ../../../knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#13 0x00007f7895260ae6 in KNS3::Engine::vote (this=<value optimized out>, entry=..., rating=110) at ../../../knewstuff/knewstuff3/core/engine.cpp:481
#14 0x00007f7895277f1b in KNS3::EntryDetails::ratingChanged (this=0x231d0f0, rating=55) at ../../../knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
#15 0x00007f789527a74b in KNS3::EntryDetails::qt_metacall (this=0x231d0f0, _c=QMetaObject::InvokeMetaMethod, _id=<value optimized out>, _a=0x7fffdd36d310) at ./entrydetailsdialog.moc:98
*** Bug 267452 has been marked as a duplicate of this bug. *** *** Bug 268392 has been marked as a duplicate of this bug. *** Created attachment 57944 [details]
New crash information added by DrKonqi
systemsettings (1.0) on KDE Platform 4.6.00 (4.6.0) "release 6" using Qt 4.7.1
- What I was doing when the application crashed:
Browsing for a new icon theme, and when I selected details on the Crystal Project, System Settings crashed.
-- Backtrace (Reduced):
#11 0x00007fbf40d62c4a in qt_assert (assertion=0x49e0 <Address 0x49e0 out of bounds>, file=0x6 <Address 0x6 out of bounds>, line=-1) at global/qglobal.cpp:2027
#12 0x00007fbf3faa2618 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib64/libattica.so.0
#13 0x00007fbf2ef95603 in KNS3::AtticaProvider::vote (this=0x8fd330, entry=<value optimized out>, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#14 0x00007fbf2efa34cd in KNS3::Engine::vote (this=<value optimized out>, entry=..., rating=110) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/core/engine.cpp:481
#15 0x00007fbf2efa3562 in KNS3::EntryDetails::ratingChanged (this=0xaf6100, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
*** Bug 268676 has been marked as a duplicate of this bug. *** *** Bug 269521 has been marked as a duplicate of this bug. *** *** Bug 269062 has been marked as a duplicate of this bug. *** *** Bug 270070 has been marked as a duplicate of this bug. *** Created attachment 58583 [details]
New crash information added by DrKonqi
plasma-desktop (0.4) on KDE Platform 4.6.1 (4.6.1) using Qt 4.7.2
- What I was doing when the application crashed:
plasma-workspace crashed when i browsing new widgets and looked into details of one of sensors widgets
-- Backtrace (Reduced):
#11 0x00007f7ce349cac0 in Attica::Provider::voteForContent(QString const&, unsigned int) () from /usr/lib64/libattica.so.0
#12 0x00007f7cd7d27c50 in KNS3::AtticaProvider::vote(KNS3::EntryInternal const&, unsigned int) () from /usr/lib64/libknewstuff3.so.4
#13 0x00007f7cd7d03c92 in KNS3::Engine::vote(KNS3::EntryInternal const&, unsigned int) () from /usr/lib64/libknewstuff3.so.4
#14 0x00007f7cd7d1bbb2 in KNS3::EntryDetails::ratingChanged(unsigned int) () from /usr/lib64/libknewstuff3.so.4
#15 0x00007f7cd7d1e1fb in KNS3::EntryDetails::qt_metacall(QMetaObject::Call, int, void**) () from /usr/lib64/libknewstuff3.so.4
*** Bug 270488 has been marked as a duplicate of this bug. *** *** Bug 270619 has been marked as a duplicate of this bug. *** *** Bug 271033 has been marked as a duplicate of this bug. *** *** Bug 272113 has been marked as a duplicate of this bug. *** *** Bug 271881 has been marked as a duplicate of this bug. *** *** Bug 272478 has been marked as a duplicate of this bug. *** *** Bug 274366 has been marked as a duplicate of this bug. *** *** Bug 274810 has been marked as a duplicate of this bug. *** The last duplicate suggests that this bug is still present in 4.6.2 To Beat Wolf: No. The problem is in libattica. Update libattica. I test it now with the latest dev libattica. And it works. So, this bug is really resolved. Created attachment 61692 [details]
New crash information added by DrKonqi
plasma-desktop (0.4) on KDE Platform 4.6.00 (4.6.0) "release 6" using Qt 4.7.1
- What I was doing when the application crashed: searching to add a widget, I found one, then I clicked "details" button and it crashed. Usually happens... but not always.
-- Backtrace (Reduced):
#11 0x00007f95ca04fc4a in qt_assert (assertion=0xfa9 <Address 0xfa9 out of bounds>, file=0x6 <Address 0x6 out of bounds>, line=-1) at global/qglobal.cpp:2027
#12 0x00007f95c8219618 in Attica::Provider::voteForContent (this=0x1b72f78, contentId=..., rating=110) at /usr/src/debug/attica-0.2.0/lib/provider.cpp:601
#13 0x00007f95be108603 in KNS3::AtticaProvider::vote (this=0x1b72f30, entry=<value optimized out>, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#14 0x00007f95be1164cd in KNS3::Engine::vote (this=<value optimized out>, entry=..., rating=110) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/core/engine.cpp:481
#15 0x00007f95be116562 in KNS3::EntryDetails::ratingChanged (this=0x1bc0210, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
*** Bug 277851 has been marked as a duplicate of this bug. *** Hello Christoph Feck: I want to say thank you for your responding to my bug-control request.. with a working solution no less. John Ellis On 07/15/2011 11:34 PM, Christoph Feck wrote: > https://bugs.kde.org/show_bug.cgi?id=261530 > > > Christoph Feck<christoph@maxiom.de> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > Component|knewstuff |general > Version|SVN |unspecified > Product|kdelibs |attica > > > > *** Bug 277849 has been marked as a duplicate of this bug. *** I don't think the problem is in libattica. The first problem is that there is a voting while I haven't given any vote. I've updated the libattica to latest dev, the browsing doesn't crash, but there is still a message, which thanks me for my vote. I haven't made any vode, I've clicked the "Details" button. *** Bug 280492 has been marked as a duplicate of this bug. *** I've tracked the problem down. Everything is in the backtrace (see below for explanation). Please note that normally the EntryInternal.rating() has rating from web - 0-100. 1. KNS3::DownloadWidgetPrivate::slotShowDetails() - Details button has been selected 2. KNS3::EntryDetails::setEntry() calls entryChanged() 3. KNS3::EntryDetails::entryChanged() gets the rating from EntryInternal (0-100), subtracts 20 and divides it by 6 and sends it to KRatingWidget. 4. KNS3::EntryDetails::ratingChanged() is called with the magic rating number, which is sent to libattica. So the libattica isn't wrong, it does great job (and doesn't need updated). KNewStuff3 is wrong - it makes voting from some magic number, but it should just show it only on KRatingWidget. I've created review #102416 with patch that fixes the problem on the place where it really happened. Can anybody change the "Product" field to kdelibs? The problem is in knewstuff3. Created attachment 63076 [details]
Patch to fix the voting problem
Git commit d2dd87af8b6bd2146b4e7434625738687a2130f8 by Jeremy Whiting. Committed on 24/08/2011 at 04:36. Pushed by whiting into branch 'frameworks'. Fix rating in knewstuff3 BUG: 261530 REVIEW: 102416 (cherry picked from commit 4da76baa3261ca2d98a495eeecc1d1c386f2e8f2) M +4 -2 knewstuff/knewstuff3/ui/entrydetailsdialog.cpp http://commits.kde.org/kdelibs/d2dd87af8b6bd2146b4e7434625738687a2130f8 Git commit 37a5ca1ec9565f6b45de7c5b12bea4bcec8d7281 by Jeremy Whiting. Committed on 24/08/2011 at 04:36. Pushed by whiting into branch 'KDE/4.7'. Fix rating in knewstuff3 BUG: 261530 REVIEW: 102416 M +4 -2 knewstuff/knewstuff3/ui/entrydetailsdialog.cpp http://commits.kde.org/kdelibs/37a5ca1ec9565f6b45de7c5b12bea4bcec8d7281 *** Bug 280917 has been marked as a duplicate of this bug. *** *** Bug 281639 has been marked as a duplicate of this bug. *** Created attachment 63831 [details]
New crash information added by DrKonqi
systemsettings (1.0) on KDE Platform 4.6.00 (4.6.0) "release 6" using Qt 4.7.1
- What I was doing when the application crashed:
Browsing the Get More Themes with the KStarBoard recently installed. I clicked "Details" or "Install" for a theme and the crash occured.
-- Backtrace (Reduced):
#11 0x00007f7a22a24c4a in qt_assert (assertion=0x5464 <Address 0x5464 out of bounds>, file=0x6 <Address 0x6 out of bounds>, line=-1) at global/qglobal.cpp:2027
#12 0x00007f7a21764618 in Attica::Provider::voteForContent (this=0x12258f8, contentId=..., rating=110) at /usr/src/debug/attica-0.2.0/lib/provider.cpp:601
#13 0x00007f7a0dcb0603 in KNS3::AtticaProvider::vote (this=0x12258b0, entry=<value optimized out>, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/attica/atticaprovider.cpp:330
#14 0x00007f7a0dcbe4cd in KNS3::Engine::vote (this=<value optimized out>, entry=..., rating=110) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/core/engine.cpp:481
#15 0x00007f7a0dcbe562 in KNS3::EntryDetails::ratingChanged (this=0xe014f0, rating=<value optimized out>) at /usr/src/debug/kdelibs-4.6.0/knewstuff/knewstuff3/ui/entrydetailsdialog.cpp:300
*** Bug 283110 has been marked as a duplicate of this bug. *** *** Bug 285136 has been marked as a duplicate of this bug. *** *** Bug 285446 has been marked as a duplicate of this bug. *** *** Bug 286154 has been marked as a duplicate of this bug. *** *** Bug 290821 has been marked as a duplicate of this bug. *** *** Bug 292809 has been marked as a duplicate of this bug. *** |