Bug 496175

Summary:	Assert in 7zip on empty file name
Product:	[Frameworks and Libraries] frameworks-karchive	Reporter:	Kai Uwe Broulik <kde>
Component:	general	Assignee:	KIO Bugs <kio-bugs-null>
Status:	RESOLVED FIXED
Severity:	crash	CC:	azhar-momin, kdelibs-bugs-null
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/karchive/-/commit/5267fb7c79fda68e589757ddea77433db8c1f932	Version Fixed/Implemented In:
Sentry Crash Report:

Description Kai Uwe Broulik 2024-11-12 14:16:17 UTC

SUMMARY

STEPS TO REPRODUCE
1. Download https://mirrors.xtom.de/kde-application/craft/Qt6/24.04/macos/clang/arm64/RelWithDebInfo/qt-libs/poppler/poppler-23.09.0-1-537-20240410T230430-macos-clang-arm64.tar.7z
2. Try to open it in sevenz: KIO through e.g. dolphin

OBSERVED RESULT
ASSERT in !entryName.isEmpty() in k7zip.cpp:2684

EXPECTED RESULT
File opens fine or shows an error but doesn’t run into an assert

SOFTWARE/OS VERSIONS
KDE Plasma Version: 6.2.4
KDE Frameworks Version: git master as of 2024-11-11 
Qt Version: 6.8.0

ADDITIONAL INFORMATION

Comment 1 Azhar Momin 2025-02-01 14:51:04 UTC

Not able to repro this error when extracting your given zip file. Was this fixed?

KDE Plasma Version: 6.2.5
KDE Frameworks Version: 6.10.0
Qt Version: 6.8.1

Comment 2 Albert Astals Cid 2025-02-27 08:57:28 UTC

Git commit 5267fb7c79fda68e589757ddea77433db8c1f932 by Albert Astals Cid, on behalf of Azhar Momin.
Committed on 27/02/2025 at 08:57.
Pushed by aacid into branch 'master'.

7zip: Add default naming for files without a name

A  +-    --    autotests/data/multiple_nameless_files.7z
M  +46   -1    autotests/karchivetest.cpp
M  +2    -0    autotests/karchivetest.h
M  +22   -4    src/k7zip.cpp

https://invent.kde.org/frameworks/karchive/-/commit/5267fb7c79fda68e589757ddea77433db8c1f932