[SCM] Samba Shared Repository - branch master updated
Jelmer Vernooij
jelmer at debian.org
Fri May 28 13:38:25 MDT 2010
Hi Matthias,
On Fri, 2010-05-28 at 12:08 -0500, Matthias Dieter Wallnöfer wrote:
> Summary of changes:
> source4/web_server/wsgi.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
>
> Changeset truncated at 500 lines:
>
> diff --git a/source4/web_server/wsgi.c b/source4/web_server/wsgi.c
> index 7a23ae4..db1a5f2 100644
> --- a/source4/web_server/wsgi.c
> +++ b/source4/web_server/wsgi.c
> @@ -301,7 +301,8 @@ static PyObject *create_environ(bool tls, int content_length, struct http_header
> PyDict_SetItemString(env, "CONTENT_TYPE", PyString_FromString(hdr->value));
> } else {
> if (asprintf(&name, "HTTP_%s", hdr->name) < 0) {
> - continue;
> + PyErr_NoMemory();
> + return NULL;
> }
This will leak the 'env' python object. You probably also want to call
Py_DECREF here.
Cheers,
Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-cvs/attachments/20100528/2d35668c/attachment.pgp>
More information about the samba-cvs
mailing list