Bug 429335

Summary: digikam 7.2.0-beta2 (18.11.2020) crashes after assigning name to unknown face
Product: [Applications] digikam Reporter: Kurpfaelzer <Stephan.Kranz>
Component: Faces-WorkflowAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: crash CC: metzpinguin
Priority: NOR    
Version: 7.2.0   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: https://invent.kde.org/graphics/digikam/commit/22e497ad75f6808df2707472ecae72da60e24710 Version Fixed In: 7.2.0
Sentry Crash Report:

Description Kurpfaelzer 2020-11-19 06:31:02 UTC 
SUMMARY
Face detection by itself is working, but after assigning the name to an "unknown face" and klicking the OK-button, digikam crashes.
After a restart of digikam, the "face" is known with the correct name which was assigned before.


SOFTWARE/OS VERSIONS
Windows: Windows 10, Version 20H2
digikam: digiKam-7.2.0-beta2-20201118T201703-Win64
Comment 1 Maik Qualmann 2020-11-19 07:08:56 UTC 
We must now use setImages(). The cause is this commit:

https://invent.kde.org/graphics/digikam/-/commit/06304b820fcee383b5a747af806c82cc44655b3b

Maik
Comment 2 Maik Qualmann 2020-11-19 07:12:01 UTC 
crash point:

trainerworker.cpp:135

Maik
Comment 3 Maik Qualmann 2020-11-19 17:46:55 UTC 
Git commit 22e497ad75f6808df2707472ecae72da60e24710 by Maik Qualmann.
Committed on 19/11/2020 at 17:46.
Pushed by mqualmann into branch 'master'.

fix crash in the TrainerWorker
FIXED-IN: 7.2.0

M  +2    -1    NEWS
M  +4    -3    core/utilities/facemanagement/workers/trainerworker.cpp

https://invent.kde.org/graphics/digikam/commit/22e497ad75f6808df2707472ecae72da60e24710
Comment 4 Maik Qualmann 2020-11-19 17:58:32 UTC 
Git commit 7b74bbd030dfb7292461aa8ac3d80a59d4e2fb9b by Maik Qualmann.
Committed on 19/11/2020 at 17:57.
Pushed by mqualmann into branch 'master'.

fix memory leak

M  +5    -0    core/utilities/facemanagement/workers/trainerworker.cpp

https://invent.kde.org/graphics/digikam/commit/7b74bbd030dfb7292461aa8ac3d80a59d4e2fb9b
Comment 5 Kurpfaelzer 2020-11-22 09:21:03 UTC 
The problem has been solved. THX to all.
Stephan