[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-05-04 Thread Paulo
https://bugs.kde.org/show_bug.cgi?id=436181

--- Comment #9 from Paulo  ---
Hey Albert thank you very much, I just downloaded and built kpat-master and it
seems that it's fixed. Moving aces doesn't crash kpat anymore.
Output of 'kpat --version' is 'kpat 21.7.70'
Regards Paulo.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-05-04 Thread Albert Astals Cid
https://bugs.kde.org/show_bug.cgi?id=436181

Albert Astals Cid  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/game |https://invent.kde.org/game
   |s/kpat/commit/d2372a8c2449a |s/kpat/commit/638d8c2c6b470
   |ee6568148cf8b03bd8acd9aa025 |54bcf3deece3b065ac5c2fb01d8

--- Comment #8 from Albert Astals Cid  ---
Git commit 638d8c2c6b47054bcf3deece3b065ac5c2fb01d8 by Albert Astals Cid, on
behalf of Michael Lang.
Committed on 04/05/2021 at 22:23.
Pushed by aacid into branch 'release/21.04'.

Fix ace auto move crash

Ace foundation auto moves were crashing after the game type was changed. Now
removing PatPile before removing Pile during game type change.
(cherry picked from commit d2372a8c2449aee6568148cf8b03bd8acd9aa025)

M  +10   -0src/dealer.cpp
M  +1-0src/dealer.h
M  +3-11   src/freecell.cpp

https://invent.kde.org/games/kpat/commit/638d8c2c6b47054bcf3deece3b065ac5c2fb01d8

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-05-04 Thread Albert Astals Cid
https://bugs.kde.org/show_bug.cgi?id=436181

Albert Astals Cid  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED
  Latest Commit||https://invent.kde.org/game
   ||s/kpat/commit/d2372a8c2449a
   ||ee6568148cf8b03bd8acd9aa025

--- Comment #7 from Albert Astals Cid  ---
Git commit d2372a8c2449aee6568148cf8b03bd8acd9aa025 by Albert Astals Cid, on
behalf of Michael Lang.
Committed on 04/05/2021 at 22:21.
Pushed by aacid into branch 'master'.

Fix ace auto move crash

Ace foundation auto moves were crashing after the game type was changed. Now
removing PatPile before removing Pile during game type change.

M  +10   -0src/dealer.cpp
M  +1-0src/dealer.h
M  +3-11   src/freecell.cpp

https://invent.kde.org/games/kpat/commit/d2372a8c2449aee6568148cf8b03bd8acd9aa025

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-04-27 Thread Paulo
https://bugs.kde.org/show_bug.cgi?id=436181

--- Comment #6 from Paulo  ---
(In reply to Albert Astals Cid from comment #5)
> Ah, Forecell isn't a typo and it's actually a game type :D
> 
> Sorry, yes i can confirm it crashes here too.

Yes :D try any Freecell variant, it should certainly (unfortunately) crash too.

Those Freecell variants were added with the new KDE applications update 21.04,
that's why I think maybe there is something related.
Before those variants addition, Freecell was running ok.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-04-27 Thread Albert Astals Cid
https://bugs.kde.org/show_bug.cgi?id=436181

Albert Astals Cid  changed:

   What|Removed |Added

 Status|NEEDSINFO   |CONFIRMED
 Ever confirmed|0   |1
 Resolution|WAITINGFORINFO  |---

--- Comment #5 from Albert Astals Cid  ---
Ah, Forecell isn't a typo and it's actually a game type :D

Sorry, yes i can confirm it crashes here too.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-04-26 Thread Paulo
https://bugs.kde.org/show_bug.cgi?id=436181

--- Comment #4 from Paulo  ---
This?
https://github.com/Ponce/slackbuilds/tree/current/desktop/flatpak

After some commands to download and install things, I think I did it right.
I could run kpat with 'flatpak run org.kde.kpat', it started with defaults
instead my own configurations.
It crashed the same way, I started a Freecell game then changed it to Forecell.
Any card auto-moves with right-click, but right-click on any ace crashes.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-04-26 Thread Albert Astals Cid
https://bugs.kde.org/show_bug.cgi?id=436181

--- Comment #3 from Albert Astals Cid  ---
That's sadly not very useful.

Does slackware have the hability to run flatpak?

If so could you try if using https://flathub.org/apps/details/org.kde.kpat
still crashes for you?

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-04-26 Thread Paulo
https://bugs.kde.org/show_bug.cgi?id=436181

--- Comment #2 from Paulo  ---
Hi Albert, thanks for your reply.
It crashes every time here :/ I also tested in a VirtualBox vm clean install
to make sure and kpat crashes too. I didn't test with another distro.

Slackware strips debug symbols from its packages,
is it enough to build qt5 without strip?

There is a valgrind package for Slackware
https://github.com/Ponce/slackbuilds/tree/current/development/valgrind
is that right?

I'm a pt_BR user so I ran as 'LANG=C valgrind kpat' to translate some messages.
That's the terminal output:

$ LANG=C valgrind kpat
==6008== Memcheck, a memory error detector
==6008== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==6008== Using Valgrind-3.17.0 and LibVEX; rerun with -h for copyright info
==6008== Command: kpat
==6008== 
==6008== Conditional jump or move depends on uninitialised value(s)
==6008==at 0x10A7ABD5: ???
==6008==by 0xC78DAA7: ???
==6008== 
==6008== Conditional jump or move depends on uninitialised value(s)
==6008==at 0x10A7ABD5: ???
==6008==by 0xC78EDC7: ???
==6008== 
==6008== Conditional jump or move depends on uninitialised value(s)
==6008==at 0x10A7ABD5: ???
==6008==by 0xC790047: ???
==6008== 
==6008== Invalid read of size 8
==6008==at 0x61E4B6E: QGraphicsScene::addItem(QGraphicsItem*) (in
/usr/lib64/libQt5Widgets.so.5.15.2)
==6008==by 0x491E972: KCardPile::insert(int, KCard*) (in
/usr/lib64/libkcardgame.so)
==6008==by 0x4921BA3: ??? (in /usr/lib64/libkcardgame.so)
==6008==by 0x4922809: KCardScene::moveCardsToPile(QList const&,
KCardPile*, int) (in /usr/lib64/libkcardgame.so)
==6008==by 0x4924005: KCardScene::moveCardToPile(KCard*, KCardPile*, int)
(in /usr/lib64/libkcardgame.so)
==6008==by 0x427BBB: ??? (in /usr/bin/kpat)
==6008==by 0x464B68: ??? (in /usr/bin/kpat)
==6008==by 0x6DFBAF3: ??? (in /usr/lib64/libQt5Core.so.5.15.2)
==6008==by 0x4916D51: KCardScene::cardRightClicked(KCard*) (in
/usr/lib64/libkcardgame.so)
==6008==by 0x4924ABC:
KCardScene::mouseReleaseEvent(QGraphicsSceneMouseEvent*) (in
/usr/lib64/libkcardgame.so)
==6008==by 0x42AFCE: ??? (in /usr/bin/kpat)
==6008==by 0x61EFC61: QGraphicsScene::event(QEvent*) (in
/usr/lib64/libQt5Widgets.so.5.15.2)
==6008==  Address 0x8 is not stack'd, malloc'd or (recently) free'd
==6008== 
KCrash: crashing... crashRecursionCounter = 2
KCrash: Application Name = kpat path = /usr/bin pid = 6008
KCrash: Arguments: /usr/bin/kpat 
KCrash: Attempting to start /usr/lib64/drkonqi

[1]+  Parado  LANG=C valgrind kpat
paulo@paulobash~$ QSocketNotifier: Invalid socket 15 and type 'Read',
disabling...
QSocketNotifier: Invalid socket 17 and type 'Read', disabling...

(kpat:6008): GLib-GIO-ERROR **: 10:02:44.704: inotify read(): Bad file
descriptor
==6008== 
==6008== HEAP SUMMARY:
==6008== in use at exit: 26,164,098 bytes in 58,034 blocks
==6008==   total heap usage: 410,092 allocs, 352,058 frees, 7,301,726,375 bytes
allocated
==6008== 
==6008== LEAK SUMMARY:
==6008==definitely lost: 1,824 bytes in 57 blocks
==6008==indirectly lost: 0 bytes in 0 blocks
==6008==  possibly lost: 318,759 bytes in 46 blocks
==6008==still reachable: 25,770,995 bytes in 57,269 blocks
==6008==   of which reachable via heuristic:
==6008== newarray   : 24 bytes in 1 blocks
==6008== multipleinheritance: 3,336 bytes in 3 blocks
==6008== suppressed: 0 bytes in 0 blocks
==6008== Rerun with --leak-check=full to see details of leaked memory
==6008== 
==6008== Use --track-origins=yes to see where uninitialised values come from
==6008== For lists of detected and suppressed errors, rerun with: -s
==6008== ERROR SUMMARY: 4 errors from 4 contexts (suppressed: 0 from 0)

[1]+  Fim da execução com status 253LANG=C valgrind kpat

-- 
You are receiving this mail because:
You are watching all bug changes.

[kpat] [Bug 436181] kpat crashes when right-click an ace card to send it upper right position.

2021-04-25 Thread Albert Astals Cid
https://bugs.kde.org/show_bug.cgi?id=436181

Albert Astals Cid  changed:

   What|Removed |Added

 CC||aa...@kde.org
 Resolution|--- |WAITINGFORINFO
 Status|REPORTED|NEEDSINFO

--- Comment #1 from Albert Astals Cid  ---
It works here :/

can you run kpat under valgrind and make what you do to make it crash and
attach here the valgrind log?

that is, run
  valgrind kpat
in a terminal (you may need to install valgrind first). Also be patient,
everything will be much slower.

Also not sure if slackware has debug symbols, but if you could install the
debug symbol packages for libQt5Widgets.so.5 it'd be best :)

-- 
You are receiving this mail because:
You are watching all bug changes.