https://bugs.kde.org/show_bug.cgi?id=466229

            Bug ID: 466229
           Summary: Spider segfaulting, new solver (417bdc2ec) bug?
    Classification: Applications
           Product: kpat
           Version: unspecified
          Platform: Gentoo Packages
                OS: Linux
            Status: REPORTED
          Severity: crash
          Priority: NOR
         Component: solver
          Assignee: co...@kde.org
          Reporter: 1i5t5.dun...@cox.net
                CC: kde-games-b...@kde.org
  Target Milestone: ---

Live-git version updated yesterday (Feb 20, 2023), using the gentoo/kde project
overlay's live-git packages, reported as 23.03.70 (both unlisted so I can't set
that version above), with live-git frameworks-5 as well.  Qt is 5.15.8 (current
as of yesterday with gentoo's regular sub-version update patchsets) .

Spider has started segfaulting on me recently (the other kpat games I play
regularly, klondike and freecell, are fine), with several weeks of further
updates not fixing it yet.  The timing and spider-specificness suggest it's due
to a race or nul-deref in the new solver, tho I'm filing this with the info I
have before bisecting to confirm that.

Maybe happenstance, but at first it /seemed/ to happen nearly immediately after
starting a spider round, often on (before ?) the first move, but something
(maybe just another rebuild, or an update of libkdegames, some framework, or
qt5, as I see no further kpat code changes gitlogged, only l10n/appstream)
seems to have decreased the frequency of the issue since and I can often get in
a number of moves now before the segfault.  Once I was even able to finish a
round and I hoped for a moment the problem was fixed, but it segfaulted on the
second round.

The output when started from konsole is unhelpful; an initial complaint from
QFont::setPixelSize that appears unrelated as it happens well before the
segfault, then nothing as I play until the segfault:

QFont::setPixelSize: Pixel size <= 0 (0)
Segmentation fault


For reference here's the suspect commit (email addresses despammed for
posting):

* commit 417bdc2ec
| Author:     Stephan Kulow <stephan@despammed>
| AuthorDate: Fri Jan 20 10:51:07 2023 +0100
| Commit:     Albert Astals Cid <aacid@despammed>
| CommitDate: Sun Jan 22 22:33:51 2023 +0000
|
|     Replace spider solver with a self serving solution
|
|     This is something I had lying around (don't ask). It's generally
|     slower (as it tries harder to find an optimal solution), but finds
|     solutions more reliably - and is possibly easier to debug for
|     other people :)

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

Reply via email to