Segmentation fault (core dumped). Using C module in python

Question

I am newbie in python and I am trying to launch python script with a module writen on C. I am getting Segmentation fault (core dumped) error when I am trying to launch python script. Here is a C code:

// input_device.c  
#include "Python.h"

#include "input.h"

static PyObject* input_device_open(PyObject* self, PyObject* id)
{
    int fd, nr;
    PyObject* pyfd;

    if (!PyInt_Check(id))
        return NULL;

    nr = (int)PyInt_AsLong(id);
    fd = device_open(nr, 0);
    if (fd == -1)
        return NULL;
    pyfd = PyInt_FromLong(fd);
    Py_INCREF(pyfd);
    return pyfd;
}

static PyMethodDef module_methods[] =
{
    { "device_open", (PyCFunction)input_device_open, METH_VARARGS, "..." },
    { NULL, NULL, 0, NULL }
};

PyMODINIT_FUNC initinput_device(void)
{
    Py_InitModule4("input_device", module_methods, "wrapper methods", 0, PYTHON_API_VERSION);
}

and the python script:

from input_device import device_open
device_open(1)

Could someone take a look and point me in the right direction, what I am doing wrong. Thanks in advance.

Brandon Rhodes · Accepted Answer · 2013-04-07 15:26:48Z

3

Is it legitimate to return NULL without setting an exception, or making sure that one has been set by a function you have called? I thought that NULL was a signal that Python could go look for an exception to raise for the user.

I am not sure that the Py_INCREF(pyfd); is necessary; doesn't the object already have a refcount of 1 upon creation?

edited Apr 7, 2013 at 15:26

answered Apr 5, 2013 at 11:29

Brandon Rhodes

91k16 gold badges110 silver badges149 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Wichert Akkerman Over a year ago

I don't think returning NULL is valid. You will either need to raise an exception or return Py_None (don't forget to Py_INCREF(Py_None) !).

Janne Karila · Accepted Answer · 2013-04-05 13:36:18Z

0

Your function receives a tuple of arguments. You need to extract the integer from the tuple:

static PyObject* input_device_open(PyObject* self, PyObject* args)
{
    int fd, nr;
    PyObject* pyfd;

    if (!PyArg_ParseTuple(args, "i", &nr))
        return NULL;

edited Apr 5, 2013 at 13:36

answered Apr 5, 2013 at 13:01

Janne Karila

25.3k6 gold badges59 silver badges97 bronze badges

Collectives™ on Stack Overflow

Segmentation fault (core dumped). Using C module in python

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related