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: 64-bit bug on AIX
Type: Stage:
Components: Interpreter Core Versions: Python 2.2
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: nnorwitz Nosy List: florent_cayre, nnorwitz
Priority: normal Keywords:

Created on 2002-11-18 08:17 by florent_cayre, last changed 2022-04-10 16:05 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
dynload_aix_patch florent_cayre, 2002-11-18 08:17 Patch proposal for dynload_aix.c in Python 2.2.1
dynload_aix.c florent_cayre, 2002-11-20 06:59 My dynload_aix.c file (patched version, from python 2.2.1 source)
Messages (6)
msg13338 - (view) Author: Florent Cayr� (florent_cayre) Date: 2002-11-18 08:17
A bug in dynload_aix.c causes unpredictable
segmentation faults on 64-bit AIX when importing a
module.
This bug is due to an (unsigned int) improper cast in
the aix_getoldmodules function, returning a bad address
as the beginning of the next ld_info structure pointed
to by ldiptr.
Following is a minimalist patch proposal obtained by
changing these (unsigned int) casts into (char*) casts
in the Python 2.2.1 version.

Regards,

Florent Cayré.
msg13339 - (view) Author: Neal Norwitz (nnorwitz) * (Python committer) Date: 2002-11-19 19:04
Logged In: YES 
user_id=33168

Florent, could you please point out the exact code and
change necessary?
msg13340 - (view) Author: Florent Cayr� (florent_cayre) Date: 2002-11-20 06:59
Logged In: YES 
user_id=648804

Not sure I understand your question...
However I filled in the Group drop down list and join my
entire dynload_aix.c file with the proposed correction in
it.

Regards,

Florent Cayré.
msg13341 - (view) Author: Neal Norwitz (nnorwitz) * (Python committer) Date: 2002-11-22 14:48
Logged In: YES 
user_id=33168

Florent, thank you.  Your patch seems correct, but
unfortunately I can't test it at this time.  Once I can test
it, I will check in the changes.  If someone else can verify
this patch works for them on both 32 & 64 bit compiles, I
will check it in.
msg13342 - (view) Author: Neal Norwitz (nnorwitz) * (Python committer) Date: 2002-12-31 00:08
Logged In: YES 
user_id=33168

Checked in as Python/dynload_aix.c  2.13.

2.3a1 should be out in about a day.  Florent, it would be
great if you can test it on both 32- and 64- bit AIX.
Leaving open to remind to backport to 2.2.3
msg13343 - (view) Author: Neal Norwitz (nnorwitz) * (Python committer) Date: 2003-01-10 23:36
Logged In: YES 
user_id=33168

Checked in as:
 Python/dynload_aix.c 2.11.6.1
 Misc/NEWS 1.337.2.4.2.53
History
Date User Action Args
2022-04-10 16:05:54adminsetgithub: 37493
2002-11-18 08:17:09florent_cayrecreate