This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 05/28] py-cmd.c error-checking bug fix
- From: Tom Tromey <tromey at redhat dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Wed, 15 May 2013 10:20:46 -0600
- Subject: Re: [PATCH 05/28] py-cmd.c error-checking bug fix
- References: <87ehe638ww dot fsf at fleche dot redhat dot com> <87fvym1tmo dot fsf at fleche dot redhat dot com> <5193B15C dot 8080907 at redhat dot com>
>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:
Pedro> On 04/19/2013 03:29 PM, Tom Tromey wrote:
>> diff --git a/gdb/python/py-cmd.c b/gdb/python/py-cmd.c
>> index 3da9960..ca9e415 100644
>> --- a/gdb/python/py-cmd.c
>> +++ b/gdb/python/py-cmd.c
>> @@ -668,6 +668,8 @@ gdbpy_string_to_argv (PyObject *self, PyObject *args)
>> return NULL;
>>
>> py_argv = PyList_New (0);
>> + if (py_argv == NULL)
>> + return NULL;
>>
Pedro> Shouldn't this be 'malloc_failure (0)' instead of returning NULL?
In the Python layer we follow Python conventions for error handling.
Here, PyList_New failed and set the Python exception.
We just propagate that to our caller.
Tom