➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Carbon.FSSpec.as_pathname() crashes
Type: Stage:
Components: Extension Modules Versions:
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: jackjansen Nosy List: jackjansen, mwh
Priority: release blocker Keywords:

Created on 2005-07-11 14:12 by mwh, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Messages (3)
msg25795 - (view) Author: Michael Hudson (mwh) (Python committer) Date: 2005-07-11 14:12 
There's something peculiar in the land of bgen-ed wrappers:

$ ./python.exe 
Python 2.5a0 (#1, Jul 11 2005, 13:21:11) 
[GCC 3.3 20030304 (Apple Computer, Inc. build 1671)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import Carbon.File
[45313 refs]
>>> Carbon.File.FSSpec(os.curdir).as_pathname()    
Segmentation fault

("make test" also crashes).

This is on 10.3.9.

My first investigations with gdb didn't reveal anything that made 
much sense, so it *might* be a compiler bug.  At any rate, it didn't 
do this a few weeks ago...
msg25796 - (view) Author: Jack Jansen (jackjansen) * (Python committer) Date: 2005-07-12 20:18 
Logged In: YES 
user_id=45365

The problem appears to be in as_pathname(). Investigating...
msg25797 - (view) Author: Jack Jansen (jackjansen) * (Python committer) Date: 2005-07-12 21:26 
Logged In: YES 
user_id=45365

Argh! It turns out that patch #1035255 was incomplete: it patched 
_Filemodule.c, but not filesupport.py (bad Bob, no cookie:-)

So, when I regenerated _Filemodule.c last week FSSpec.as_pathname() 
still called FSSpec_as_pathname, in stead of _FSSpec_as_pathname 
(note the subtle difference, which I consequently overlooked). This 
resulted in an infinite loop.

Fixed in _Filemodule.c rev. 1.25, filesupport.py rev. 1.22.
History
Date User Action Args
2022-04-11 14:56:12adminsetgithub: 42182
2005-07-11 14:12:42mwhcreate