GpgFrontend/gpgme
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

python: Fix error handling.

Browse Source 
* lang/python/gpgme.i: Fix freeing an uninitialized pointer in the
error handling of generated wrapper functions by explicitly storing
the pointer in a local variable which can be initialized.

Signed-off-by: Justus Winter <justus@g10code.com>
This commit is contained in:
Justus Winter 2016-06-07 16:07:33 +02:00
parent 8196edf9ca
commit f8f9bf06bc
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lang/python/gpgme.i
View File

@ -1,4 +1,5 @@
/* /*
# Copyright (C) 2016 g10 Code GmbH
# Copyright (C) 2004,2008 Igor Belyi <belyi@users.sourceforge.net> # Copyright (C) 2004,2008 Igor Belyi <belyi@users.sourceforge.net>
# Copyright (C) 2002 John Goerzen <jgoerzen@complete.org> # Copyright (C) 2002 John Goerzen <jgoerzen@complete.org>
# #
@ -42,11 +43,11 @@
%typemap(freearg) const char * ""; %typemap(freearg) const char * "";
/* Likewise for a list of strings. */ /* Likewise for a list of strings. */
%typemap(in) const char *[] { %typemap(in) const char *[] (void *vector = NULL) {
/* Check if is a list */ /* Check if is a list */
if (PyList_Check($input)) { if (PyList_Check($input)) {
size_t i, size = PyList_Size($input); size_t i, size = PyList_Size($input);
$1 = (char **) malloc((size+1) * sizeof(char *)); $1 = (char **) (vector = malloc((size+1) * sizeof(char *)));
for (i = 0; i < size; i++) { for (i = 0; i < size; i++) {
PyObject *o = PyList_GetItem($input,i); PyObject *o = PyList_GetItem($input,i);
@ -72,7 +73,7 @@
} }
} }
%typemap(freearg) const char *[] { %typemap(freearg) const char *[] {
free((char *) $1); free(vector$argnum);
} }
// Release returned buffers as necessary. // Release returned buffers as necessary.