Bug 7053 - perl-URPM: update candidate: fix some segfaults
Summary: perl-URPM: update candidate: fix some segfaults
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 2
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: mga2-32-OK mga2-64-OK
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2012-08-14 14:18 CEST by Thierry Vignaud
Modified: 2012-09-04 21:11 CEST (History)
5 users (show)

See Also:
Source RPM: perl-URPM
CVE:
Status comment:


Attachments
diff showing one less valgrind error (1.55 KB, patch)
2012-08-14 14:19 CEST, Thierry Vignaud
Details | Diff

Description Thierry Vignaud 2012-08-14 14:18:54 CEST
A bug which resulted in rare segfault in urpmi was fixed in perl-URPM.
It happenned from time to time on the BS


Suggested advisory:
===================
A bug which resulted in rare segfault in urpmi was fixed in perl-URPM.

Steps to Reproduce:
If you really want to check, you can:
0) run "urpmi valgrind"
1) run "valgrind perl t/synthesis.t &> LOG1"
2) update perl-URPM to 3.40.2 from update_testing
3) run "valgrind perl t/synthesis.t &>LOG2"
4) run "perl -pi -e 's!^(==|--)\d+(==|--)!${1}10000\2!' LOG?"
5) run "diff -u LOG{2,1} > l.txt"
   with any editor you should see something like the diff
   attached to this BR, aka one less error
   ("Conditional jump or move depends on uninitialised value"
Comment 1 Thierry Vignaud 2012-08-14 14:19:27 CEST
Created attachment 2643 [details]
diff showing one less valgrind error
Comment 2 Thierry Vignaud 2012-08-14 14:19:45 CEST
For the record, the whole testsuite of perl-URPM & urpmi pass
Comment 3 Pascal Terjan 2012-08-14 16:52:58 CEST
Please wait a bit before testing as there is another bug fix (to write :) ) I'd like to include in this update.

CC: (none) => pterjan

Comment 4 Samuel Verschelde 2012-08-15 14:30:00 CEST
Ok, please remove the feedback marker in whiteboard once you're done.

CC: (none) => stormi
Whiteboard: (none) => feedback

Samuel Verschelde 2012-08-20 12:22:47 CEST

Summary: update candidate: fix a rare segfault => perl-URPM: update candidate: fix a rare segfault

Comment 5 Samuel Verschelde 2012-08-20 12:44:58 CEST
Assigning back to Pascal Terjan until the update is ready.

Assignee: qa-bugs => pterjan
Whiteboard: feedback => (none)

Comment 6 Pascal Terjan 2012-08-24 02:39:22 CEST
I have uploaded a new perl-URPM, and a new urpmi using that new perl-URPM which should get rid of a quite common segfault we get on buildsystem.

Should they be part of the same update?

Assignee: pterjan => qa-bugs

Comment 7 Thierry Vignaud 2012-08-24 08:30:24 CEST
I would say yes as new perl-URPM only provides a new method that is only used by new urpmi and since this new urpmi depends on this new URPM
Comment 8 Samuel Verschelde 2012-08-24 08:42:05 CEST
Yes, better test them and push them together.
Comment 9 Samuel Verschelde 2012-08-26 15:42:41 CEST
SRPMs:
urpmi-6.48.3-2.mga2.src.rpm
perl-URPM-3.40.2-1.2.mga2.src.rpm

do you confirm?
Comment 10 Thierry Vignaud 2012-08-27 00:28:14 CEST
Yes. Though it would have been better for Pascal to bump URPM to 3.40.3.
But yes those are the good versions
Pascal Terjan 2012-08-30 09:59:22 CEST

Summary: perl-URPM: update candidate: fix a rare segfault => perl-URPM: update candidate: fix some segfaults

Comment 11 Dave Hodgins 2012-08-31 03:26:49 CEST
I think rpm should be pushed at the same time as urpmi and perl-URPM,
but can't find the bug report for it.

My understanding is that the change to rpm is in preparation for usrmove,
and several comments on the dev mailing list indicate it's working, plus
it's been working for normal usage in regular qa testing.

Should it be added to this report, or should we wait for a separate bug
report for rpm to be assigned to qa?

CC: (none) => davidwhodgins

Dave Hodgins 2012-08-31 04:38:11 CEST

Whiteboard: (none) => feedback

Comment 12 Thierry Vignaud 2012-08-31 08:26:38 CEST
It's independant so it should be another BR
Samuel Verschelde 2012-08-31 08:28:18 CEST

Whiteboard: feedback => (none)

Comment 13 claire robinson 2012-08-31 16:52:48 CEST
We've all been using this for some time now with no ill effects.

Does anybody object to validating?
Comment 14 Thierry Vignaud 2012-09-01 00:16:18 CEST
Go ahead
Comment 15 claire robinson 2012-09-01 13:52:18 CEST
Could somebody please create an update request for rpm.

Validating this one.

SRPMs:
urpmi-6.48.3-2.mga2.src.rpm
perl-URPM-3.40.2-1.2.mga2.src.rpm


Advisory
-------------
This update corrects some segfaults which occurred occasionally when using urpmi.
-------------

Could sysadmin please push from core/updates_testing to core/updates.

Thanks!

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs
Whiteboard: (none) => mga2-32-OK mga2-64-OK

Comment 16 Thomas Backlund 2012-09-04 21:11:30 CEST
Update pushed:
https://wiki.mageia.org/en/Support/Advisories/MGAA-2012-0180

Status: NEW => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.