tpc_abort for blobs: "os.exists" is wrong, code is not tested

Bug #126007 reported by Christian Theune
2
Affects Status Importance Assigned to Milestone
ZODB
Status tracked in 3.9
3.8
Fix Released
Critical
Christian Theune
3.9
Fix Released
Critical
Christian Theune

Bug Description

Two problems: the spelling "os.exists" should be os.path.exists and the code in tpc_abort in ZODB/blob.py is apparently not tested for this case.

Revision history for this message
Christian Theune (ctheune) wrote :

Btw: This can make a ZEO server crash.

Changed in zodb:
importance: Undecided → Critical
Revision history for this message
Jim Fulton (jim-zope) wrote : Re: [Bug 126007] tpc_abort for blobs: "os.exists" is wrong, code is not tested

Looks like we're missing a test.

On Jul 14, 2007, at 11:29 AM, Christian Theune wrote:

> Public bug reported:
>
> Two problems: the spelling "os.exists" should be os.path.exists and
> the
> code in tpc_abort in ZODB/blob.py is apparently not tested for this
> case.
>
> ** Affects: zodb (upstream)
> Importance: Critical
> Status: New
>
> --
> tpc_abort for blobs: "os.exists" is wrong, code is not tested
> https://bugs.launchpad.net/bugs/126007
> You received this bug notification because you are a member of Zodb-
> developers, which is the registrant for ZODB.

--
Jim Fulton mailto:<email address hidden> Python Powered!
CTO (540) 361-1714 http://www.python.org
Zope Corporation http://www.zope.com http://www.zope.org

Revision history for this message
Christian Theune (ctheune) wrote : Re: [Bug 126007] tpc_abort for blobs: "os.exists" is wrong, code is not tested

Am Montag, den 16.07.2007, 12:27 +0000 schrieb Jim Fulton:
> Looks like we're missing a test.

Jupp. I haven't had time to look deeper into this, however, I might be
able to spend some time in about two weeks.

Revision history for this message
Christian Theune (ctheune) wrote :

Fixed for 3.8.0b3.

Revision history for this message
Christian Theune (ctheune) wrote :

Fixed for 3.9a1.

Revision history for this message
Christian Theune (ctheune) wrote :

Setting as released because 3.9 currently is the trunk

Revision history for this message
Christian Theune (ctheune) wrote :

was released in b3

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.