reading a python file object in c extension crashes python

[travis ray]

Hi,

I have an extension in which a file object is created in python and
passed down to a c extension which attempts to read from it or write
to it. Writing to the file pointer seems to work okay, but reading
from it results in EBADF. It also causes python to crash on exit.

I've attached the minimal (I think) c code, python code, build script,
build log, and run log. Any and all help is greatly appreciated.

Thanks.

System: Win-XP Professional
Compiler: Vc7 (MSVS 2003)
Python: Python 2.4.1
Builder: SCons 0.96.1

P.S. Sorry for including everything in one message. I keep getting
bounced when I attach them.
### file:my_module. c -------------------------------------------------

#include <Python.h>
#define MAXLEN 128

int
my_read(char *buffer, int max, FILE *fp)
{
int n = 0;
fprintf (stdout, "[c] pre-read: errno=%d, fp=%x\n", errno, fp);
n = fread(buffer, 1, max, fp);
fprintf (stdout, "[c] post-write: errno=%d, fp=%x, count=%d,
data='%s'\n", errno, fp, n, buffer);
return n;
}

int
my_write(const char *buffer, FILE *fp)
{
int n = 0;
fprintf (stdout, "[c] pre-write: errno=%d, fp=%x, data='%s'\n",
errno, fp, buffer);
n = fprintf(fp, "%s", buffer);
fprintf (stdout, "[c] post-write: errno=%d, fp=%x, count=%d\n",
errno, fp, n);
return n;
}

static PyObject *
my_module_my_re ad(PyObject *self, PyObject *args)
{
char buffer[MAXLEN];
int count = 0;
FILE *fp = NULL;
PyObject* py_file = NULL;

memset((void*)& (buffer),0,size of(char)*MAXLEN );

if (!PyArg_ParseTu ple(args, "O", &py_file))
return NULL;

if (PyFile_Check(p y_file))
{

count = my_read((char*) &(buffer), MAXLEN, PyFile_AsFile(p y_file));
return Py_BuildValue(" (is)", count, (char*)buffer);
}
return NULL;
}

static PyObject *
my_module_my_wr ite(PyObject *self, PyObject *args)
{
char buffer[MAXLEN];
int count = 0;
FILE *fp = NULL;
PyObject* py_file = NULL;
const char* py_string;

memset((void*)& (buffer),0,size of(char)*MAXLEN );

if (!PyArg_ParseTu ple(args, "sO", &py_string, &py_file))
return NULL;

if ((PyFile_Check( py_file)) && (py_string != NULL))
{
count = my_write(py_str ing, PyFile_AsFile(p y_file));
return Py_BuildValue(" i", count);
}
return NULL;
}

static PyMethodDef MyModuleMethods[] = {
{"my_read", my_module_my_re ad, METH_VARARGS,
"read at most 128 bytes from a file"},
{"my_write", my_module_my_wr ite, METH_VARARGS,
"write a string to a file"},
{NULL,NULL,0,NU LL}
};

PyMODINIT_FUNC
initmy_module(v oid)
{
PyObject *m;
m = Py_InitModule(" my_module", MyModuleMethods );
}
### file: my_module.py ------------------------------------------------

import my_module

filename = "hello.txt"
fp1 = open(filename, "wb")
print "[py] pre-write: fp=%s" % (fp1)
result1 = my_module.my_wr ite("Hello, World!", fp1)
print "[py] result=", result1
fp1.close()

fp2 = open(filename, "rb")
print "[py] pre-read: fp=%s" % (fp2)
result2 = my_module.my_re ad(fp2)
print "[py] result=", result2
#fp2.close()
# crashes python here EBADF when fp2.close() is called
print "done"
# crashes python here when fp2.close() is commented out

### file: Sconstruct --------------------------------------------------
env = Environment()
import os

python_include_ path = os.path.join(os .path.sep,'Pyth on24','include' )
python_lib_path = os.path.join(os .path.sep,'Pyth on24','libs')
python_libs = 'python24'
my_module_name = 'my_module'
my_module_sourc es = 'my_module.c'

library = env.SharedLibra ry(my_module_na me,
my_module_sourc es,
CPPPATH=[python_include_ path],
LIBPATH=[python_lib_path],
LIBS=[python_libs])
### file: build.log ---------------------------------------------------
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
cl /nologo /I\Python24\incl ude /c my_module.c /Fomy_module.obj
my_module.c
link /nologo /dll /out:my_module.d ll /implib:my_modul e.lib
/LIBPATH:\Python 24\libs python24.lib my_module.obj
Creating library my_module.lib and object my_module.exp
scons: done building targets.
### file: run.log -----------------------------------------------------
[py] pre-write: fp=<open file 'hello.txt', mode 'wb' at 0x009B54A0>
[c] pre-write: errno=0, fp=7c38b548, data='Hello, World!'
[c] post-write: errno=0, fp=7c38b548, count=13
[py] result= 13
[py] pre-read: fp=<open file 'hello.txt', mode 'rb' at 0x009B5608>
[c] pre-read: errno=0, fp=7c38b548
[c] post-write: errno=9, fp=7c38b548, count=0, data=''
[py] result= (0, '')
done