Bug 622835

Summary:	add fix for containerless unicode float decoding
Product:	[Fedora] Fedora EPEL	Reporter:	Felix Schwarz <fschwarz>
Component:	python-simplejson	Assignee:	Luke Macken <lmacken>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	el5	CC:	a.badger, fschwarz, kylev, lmacken, pfrields
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	python-simplejson-2.0.9-2.el5	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-08-30 17:54:47 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 Felix Schwarz 2010-08-10 14:20:52 UTC

In EPEL, simplejson.loads(u'-2.75') will raise an exception because of http://code.google.com/p/simplejson/issues/detail?id=43 . I'd like to request inclusion of the fix into python-simplejson for EPEL as this is a simple bug fix. The bug itself is easy to hit in 'normal' use cases.

upstream fixed this in 2.1 but that release is not officially supported on Python 2.4. Also 2.1 is in some respect incompatible to 2.0.9 so we should not update python-simplejson in general but just pull in selected fixes.

Comment 1 Toshio Kuratomi 2010-08-10 16:06:51 UTC

Looks good.  Just a question of whether that's the complete fix or not.  The commit mentioned in the bug seems to only affect the .c file used for speedup, not the .py file (which I suppose wouldn't affect us since we'll always build the speedup in Fedora packages) and the unittests don't seem to be updated to test the given error condition.

Comment 2 Felix Schwarz 2010-08-10 16:18:00 UTC

yes, the pure-python version is not affected by the bug (something that confused me thoroughly until I found out - as they seemingly similar machines showed different behavior). 

I'll try to update the package (assuming that I can beat git hard enough to get it working ;-).

Comment 3 Fedora Update System 2010-08-13 11:57:38 UTC

python-simplejson-2.0.9-2.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/python-simplejson-2.0.9-2.el5

Comment 4 Fedora Update System 2010-08-13 22:37:36 UTC

python-simplejson-2.0.9-2.el5 has been pushed to the Fedora EPEL 5 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python-simplejson'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/python-simplejson-2.0.9-2.el5

Comment 5 Fedora Update System 2010-08-30 17:54:42 UTC

python-simplejson-2.0.9-2.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.