Bug 579131
| Summary: | [abrt] crash in psi-0.14-1.fc13: Process /usr/bin/psi was killed by signal 6 (SIGABRT) | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Raphael Groner <projects.rg> | ||||||
| Component: | psi | Assignee: | Aurelien Bompard <gauret> | ||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | medium | Docs Contact: | |||||||
| Priority: | low | ||||||||
| Version: | 13 | CC: | dkovalsk, fionov, gauret, lemenkov, sven, ville.skytta | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | x86_64 | ||||||||
| OS: | Linux | ||||||||
| Whiteboard: | abrt_hash:74dc57bf9497498d322da8ed82c38a91de5eabcf | ||||||||
| Fixed In Version: | psi-0.14-3.fc13 | Doc Type: | Bug Fix | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2010-04-20 13:14:23 UTC | Type: | --- | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 496968 | ||||||||
| Attachments: |
|
||||||||
|
Description
Raphael Groner
2010-04-02 19:12:44 UTC
Created attachment 404228 [details]
File: backtrace
It seems to be related to a failing assert statement. Please rebuild the package without debug support at all. Seems that there are wrong configure options. "--debug --no-separate-debug-info" flags disable QT_NO_DEBUG macro, that makes Q_ASSERTs fatal. It is incorrect behaviour for package that is not to be used to develop Psi. When Qt debug is not enabled, all asserts are ignored. I think that "release" version should be built, with debug compiler flags enabled for debuginfo packages (and qt debug flags disabled). Workaround: Remove those three files from /usr/share/psi/iconsets: emoticons/gadu_gadu.jisp, roster/kitty.jisp and roster/dudes32.jisp Created attachment 404397 [details]
Disable Qt debugging macros
The proposed patch adds QT_NO_DEBUG define to disable Qt debugging macros. Due to "debug" config option debuginfo will not be lost.
With this patch Psi doesn't abort on incorrect iconsets as in "release" build.
psi-0.14-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/psi-0.14-2.fc13 (In reply to comment #6) > psi-0.14-2.fc13 has been submitted as an update for Fedora 13. > http://admin.fedoraproject.org/updates/psi-0.14-2.fc13 Bugfix confirmed. Above described behaviour (crash) does not occur any more. Thanks for the quick patch! *** Bug 579673 has been marked as a duplicate of this bug. *** Is there going to be an F12 update too? My F12 bug 579673 has been closed as a duplicate of this one. psi-0.14-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update psi'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/psi-0.14-2.fc13 (In reply to comment #9) > Is there going to be an F12 update too? My F12 bug 579673 has been closed as a > duplicate of this one. Sorry. I should have added a note when closing your bug. An update is also available for F12: https://admin.fedoraproject.org/updates/psi-0.14-2.fc12 Thanks Sven! :) Who is responsible for closing this bug? Can I (as the reporter) close it? I have verified the patched package is now available in repository. The bug will be closed when the package is moved to the stable repo. No need to do it yourself, but thanks anyway :) This change has made -debuginfo probably useless, I suppose (based on quick look at the build log and -debuginfo no longer having sources) because the build now strips objects before rpmbuild has had a chance to do it. See blocker bug for more info. (In reply to comment #13) I am able to test only with rawhide. Therefore and due to the duplication of bug (comments #8 to #11), I am not sure about Version and Platform. Bug seems to exist since F11. And besides x86_64 also for i686? > This change has made -debuginfo probably useless
OK, any way to keep the objects unstripped and not crash on failed assertions ?
I suggest to add QT_NO_DEBUG define which disables qt debugging macros, instead of switching off debug. See my comment #5 and patch. Based on comments 3, 4, and 18 I gather stripping is not the issue (that would be very rare), it's the other things that the --debug and --no-separate-debug-info do. No matter whether those are left in or out, it should be ensured that the upstream build respects $RPM_OPT_FLAGS and does not attempt any stripping itself, by patching if necessary. Updated package from repository is okay for me. Name : psi Architektur : x86_64 Version : 0.14 Ausgabe : 2.fc13 Grösse : 6.5 M Repo : installed Aus repo : updates-testing psi-0.14-3.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/psi-0.14-3.fc13 psi-0.14-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update psi'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/psi-0.14-3.fc13 psi-0.14-3.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. |