FS#48295 - [icu] pkgdata segfaults

Attached to Project: Arch Linux
Opened by Janne Heß (das_j) - Monday, 22 February 2016, 08:44 GMT
Last edited by Andreas Radke (AndyRTR) - Sunday, 03 July 2016, 15:56 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Andreas Radke (AndyRTR)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

When using pkgdata (comes with extra/icu), the -m option segfaults.
-m accepts the following values: files, dll, library, common, archive, static.
When using anything else than common, it segfaults without validating the other parameteres.
I can reproduce this on two Arch machines, in a nspawn container and in the archiso.
As there are no other bug reports on the internet, I suspect Arch to be the only distribution with this problem.
I created a patch and an updated PKGBUILD (added a source file and a patch call in prepare()): https://gist.github.com/dasJ/790d600902631950b7a1

Additional info:
* package version(s)

icu 56.1-2

* config and/or log files etc.

$ pkgdata -m static
[1] 25923 segmentation fault (core dumped) pkgdata -m static


Steps to reproduce:

You can either try to build LTFS from http://www.quantum.com/serviceandsupport/opensource/ltfs/index.aspx (./configure && make) or just run one of these:

pkgdata -m files
pkgdata -m dll
pkgdata -m library
pkgdata -m archive
pkgdata -m static
This task depends upon

Closed by  Andreas Radke (AndyRTR)
Sunday, 03 July 2016, 15:56 GMT
Reason for closing:  Upstream
Additional comments about closing:  fix will be included in the next upstream release.
Comment by Doug Newgard (Scimmia) - Monday, 22 February 2016, 14:44 GMT
If this requires code changes, how could it possibly be Arch specific?
Comment by Andreas Radke (AndyRTR) - Monday, 22 February 2016, 20:08 GMT
Better report it upstream and let's find out if other people and distros are also affected and if your patch is a proper solution.
Comment by Janne Heß (das_j) - Monday, 22 February 2016, 20:11 GMT
I'll check with other distros tomorrow. I don't think the patch is a solution, it's rather a workaround.
The comment before the patched macro clearly states that no null check should be required.
Comment by Andreas Radke (AndyRTR) - Saturday, 05 March 2016, 17:12 GMT
Before we close this one please file an upstream bug to the icu project tracker and leave the url here to follow.
Comment by Andreas Radke (AndyRTR) - Friday, 01 April 2016, 17:14 GMT
Have you filed an upstream report? Have you checked if this is still present when updating to icu 57.1 (we will soon start the rebuilds)?
Comment by Andreas Radke (AndyRTR) - Friday, 15 April 2016, 12:57 GMT
Please bring this upstream: http://bugs.icu-project.org/trac

I assume you've already done debug builds with logs you can provide.
Comment by Andreas Radke (AndyRTR) - Thursday, 23 June 2016, 12:44 GMT
Before I close this one as an upstream bug please open a bug in the upstream bugtracker.
Comment by Andreas Radke (AndyRTR) - Sunday, 03 July 2016, 15:55 GMT
I've tested svn trunk rev. r38937 and it is working well. So it is fixed upstream.

Loading...