Bug 593553

Summary: hdr.dsFromHeader() fails with "SystemError" for gpg-pubkeys
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: rpmAssignee: Panu Matilainen <pmatilai>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 13CC: ffesti, jnovy, pmatilai
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 593621 (view as bug list) Environment:
Last Closed: 2010-08-10 12:27:00 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Ville Skyttä 2010-05-19 06:15:29 UTC
rpm-python-4.8.0-14.fc13.x86_64 on current F-13:

import rpm
for hdr in rpm.TransactionSet().dbMatch("name", "gpg-pubkey"):
    print hdr.dsFromHeader()

Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
SystemError: NULL result without error in PyObject_Call

Slightly different message with rpm-python-4.4.2.3-18.el5 on CentOS 5:

Traceback (most recent call last):
  File "<stdin>", line 2, in ?
SystemError: error return without exception set

rpm-python bug?

This affects rpmlint (DocFilesCheck.py).  I wonder if I could just use hdr.dsOfHeader() instead of hdr.dsFromHeader() there, it doesn't seem to have the same problem (not to mention that even that doesn't have to be done for packages that don't have doc files...)

Comment 1 Panu Matilainen 2010-05-19 06:31:36 UTC
Sure it's an rpm-python bug, I'll look into it.

Comment 2 Panu Matilainen 2010-05-19 07:26:34 UTC
Fixed upstream now, pulling into Fedora shortly... Looking at rpm 4.4.x code, the actual rpmlint code works on Centos but breaks with the reproducer (its just the print blowing up), right?

hdr.dsOfHeader() and hds.dsFromHeader() are completly different things and can't be used interchangibly. dsOfHeader() returns basically
Provides: %{name} = %{version}-%{release}
whereas dsFromHeader() returns the actual requires/provides/obsoletes/conflicts dependencies.

Comment 3 Ville Skyttä 2010-05-19 14:53:03 UTC
(In reply to comment #2)
> Looking at rpm 4.4.x code,
> the actual rpmlint code works on Centos but breaks with the reproducer (its
> just the print blowing up), right?

Right.  Ditto on F-12 with rpm-python-4.7.2-1.fc12.x86_64.

Comment 4 Panu Matilainen 2010-05-21 13:29:35 UTC
Ack. Fixed in rawhide now.

Comment 5 Fedora Update System 2010-06-11 11:49:19 UTC
rpm-4.8.1-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/rpm-4.8.1-1.fc13

Comment 6 Fedora Update System 2010-07-07 17:42:37 UTC
rpm-4.8.1-2.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.