Skip to content

Commit 81ba934

Browse files
committed
Fix missing "volatile" in PLy_output().
Commit 5c3c3cd plastered "volatile" on a bunch of variables in PLy_output(), but removed the one that actually mattered, ie the one on "oldcontext". This allows some versions of clang to generate code in which "oldcontext" has been trashed when control reaches the PG_CATCH block. Per buildfarm member tick.
1 parent ee5dbc8 commit 81ba934

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/pl/plpython/plpy_plpymodule.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ PLy_output(volatile int level, PyObject *self, PyObject *args, PyObject *kw)
404404
char *volatile datatype = NULL;
405405
char *volatile table = NULL;
406406
char *volatile schema = NULL;
407-
MemoryContext oldcontext ;
407+
volatile MemoryContext oldcontext;
408408
PyObject *key, *value;
409409
PyObject *volatile so;
410410
Py_ssize_t pos = 0;

0 commit comments

Comments
 (0)