pasnox / qwbfsmanager

QWBFS Manager is a Wii games manager
62 stars 7 forks source link

Qwbfs crash when click on "Select partition" #39

Closed Nekmo closed 8 years ago

Nekmo commented 9 years ago

This program is inusable. When I click on "Set the current partition" select input, Qwbfs closes without any message error.

System: Arch Linux x86_64. Qwbfs version: 1.2.5.

pasnox commented 9 years ago

Did you started it in root ?

Nekmo commented 9 years ago

Yes. El 19/4/2015 11:36, "Nox P@sNox" notifications@github.com escribió:

Did you started it in root ?

— Reply to this email directly or view it on GitHub https://github.com/pasnox/qwbfsmanager/issues/39#issuecomment-94256418.

ReiseArnor commented 9 years ago

Me too. The same error.

System: Manjaro 15.09 32-bits

pasnox commented 9 years ago

Which Qt version ? 32 or 64bits ?

2015-10-21 7:36 GMT+02:00 odanis notifications@github.com:

Me too. The same error.

System: Manjaro 15.09 32-bits

— Reply to this email directly or view it on GitHub https://github.com/pasnox/qwbfsmanager/issues/39#issuecomment-149788513.

Nekmo commented 9 years ago

64. El 21/10/2015 10:05, "Nox P@sNox" notifications@github.com escribió:

Which Qt version ? 32 or 64bits ?

2015-10-21 7:36 GMT+02:00 odanis notifications@github.com:

Me too. The same error.

System: Manjaro 15.09 32-bits

— Reply to this email directly or view it on GitHub <https://github.com/pasnox/qwbfsmanager/issues/39#issuecomment-149788513 .

— Reply to this email directly or view it on GitHub https://github.com/pasnox/qwbfsmanager/issues/39#issuecomment-149810442.

ReiseArnor commented 9 years ago

32 bits

PeakiNPC commented 8 years ago

me too, i have archlinux 64 bits this typing in terminal:

[sen@Gundam-Zer0 ~]$ qwbfsmanager libpng warning: iCCP: known incorrect sRGB profile "Error downloading https://www.paypal.com/es_ES/i/btn/btn_donate_LG.gif - server replied: Forbidden" QUrl("https://www.paypal.com/es_ES/i/btn/btn_donate_LG.gif") [sen@Gundam-Zer0 ~]$ sudo qwbfsmanager [sudo] password for sen: libpng warning: iCCP: known incorrect sRGB profile Violación de segmento

Salamandar commented 8 years ago

Same here. I installed the binary + built myself the package.

pasnox commented 8 years ago

Will check that if i got time during the weekend

Salamandar commented 8 years ago

By the way I can't compile it a second time, it complains about the missing libfresh… But it worked the first time O.o So I could'nt compile it with debug symbols to have more info.

pasnox commented 8 years ago

Do use shadow builds, aka out of tree source builds:

This allow to keep the source folder clean so u can do many builds as u like using different build folders.

Salamandar commented 8 years ago

Yeah, that's what I did like specified in the Readme ;) I found the problem for the build : There is libfresh_debug.a in the build folder, but g++ is searching for lfresh (libfresh.a, without _debug). A manual file copy is enough to solve the problem for now.

Salamandar commented 8 years ago

Yeah, well, debug symbols don't help so much :

Program received signal SIGSEGV, Segmentation fault.
0x00000000004706e5 in PartitionDelegate::paint(QPainter*, QStyleOptionViewItem const&, QModelIndex const&) const ()
(gdb) bt
#0  0x00000000004706e5 in PartitionDelegate::paint(QPainter*, QStyleOptionViewItem const&, QModelIndex const&) const ()
Backtrace stopped: Cannot access memory at address 0x7fffffffc7c8
Salamandar commented 8 years ago

Hmmm, in fact the debug build is… not a debug build. I had to add manually "-g" to the makefiles… :-1:

Program received signal SIGSEGV, Segmentation fault.
0x00000000004706e5 in PartitionDelegate::paint (this=0x998100, painter=0x7fffffffc870, _option=..., index=...) at ../../qwbfs/src/models/PartitionDelegate.cpp:90
90      option.palette = mStyle->standardPalette();
(gdb) bt
#0  0x00000000004706e5 in PartitionDelegate::paint (this=0x998100, painter=<error reading variable: Cannot access memory at address 0x7fffffffc428>, _option=..., index=...)
    at ../../qwbfs/src/models/PartitionDelegate.cpp:90
#1  0x00007ffff76b8024 in QListView::paintEvent(QPaintEvent*) () from /usr/lib/libQt5Widgets.so.5
Backtrace stopped: Cannot access memory at address 0x7fffffffca28
Salamandar commented 8 years ago

I commented the lines 90, 162,163 in the qwbfs/src/models/PartitionDelegate.cpp, and it works, but the partition names are only visible with the drop-down menu closed AND the cursor on the drop-down menu.

pasnox commented 8 years ago

Which Qt version we speak here ? Can reproduce on OSX... will fix now but only in git (no time to create new release for now)

pasnox commented 8 years ago

Fixed in master and branch 1.2.x. Taggued 1.2.6 version.

Just build from source to avoid crash, will try to produce packages in time, but clearly don't rely on that for a while ;)

Salamandar commented 8 years ago

Got it. I don't understand the lines :

            macx:LIBS *= -lfresh_debug
            else:LIBS *= -lfreshd

I am on Linux but I need -lfresh_debug, not -lfreshd.

Salamandar commented 8 years ago

Well the fix works by me :D

pasnox commented 8 years ago

That's weird, linux default to append a d for debug libs, maybe i wrongly setted to _debug on linux for fresh, i will check. But normally release builds should be fine by now. EDIT: Indeed fresh do generate _debug for unix and d for windows. I will fix. Thanks

Salamandar commented 8 years ago

Great :D