diff options
author | Matthieu Patou <mat@matws.net> | 2011-09-22 19:31:32 -0700 |
---|---|---|
committer | Matthieu Patou <mat@samba.org> | 2011-09-23 06:06:57 +0200 |
commit | 766d43efde768427ee2ab7fe7f1f958a794e74da (patch) | |
tree | 5d87427496b1e7d76e73496733e4c511cfab56c5 /source4 | |
parent | f0f91d0117b2ccf778382e61a411b5e4f0ea2b14 (diff) | |
download | samba-766d43efde768427ee2ab7fe7f1f958a794e74da.tar.gz samba-766d43efde768427ee2ab7fe7f1f958a794e74da.tar.bz2 samba-766d43efde768427ee2ab7fe7f1f958a794e74da.zip |
pys4: really return an exception if the status if not correct
Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Fri Sep 23 06:06:57 CEST 2011 on sn-devel-104
Diffstat (limited to 'source4')
-rw-r--r-- | source4/ntvfs/posix/python/pyxattr_tdb.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/source4/ntvfs/posix/python/pyxattr_tdb.c b/source4/ntvfs/posix/python/pyxattr_tdb.c index 2b28aca365..a08557e08d 100644 --- a/source4/ntvfs/posix/python/pyxattr_tdb.c +++ b/source4/ntvfs/posix/python/pyxattr_tdb.c @@ -60,9 +60,8 @@ static PyObject *py_wrap_setxattr(PyObject *self, PyObject *args) status = push_xattr_blob_tdb_raw(eadb, mem_ctx, attribute, filename, -1, &blob); if (!NT_STATUS_IS_OK(status)) { - PyErr_FromNTSTATUS(status); talloc_free(mem_ctx); - return NULL; + PyErr_NTSTATUS_IS_ERR_RAISE(status); } talloc_free(mem_ctx); Py_RETURN_NONE; @@ -91,9 +90,8 @@ static PyObject *py_wrap_getxattr(PyObject *self, PyObject *args) status = pull_xattr_blob_tdb_raw(eadb, mem_ctx, attribute, filename, -1, 100, &blob); if (!NT_STATUS_IS_OK(status) || blob.length < 0) { - PyErr_FromNTSTATUS(status); talloc_free(mem_ctx); - return NULL; + PyErr_NTSTATUS_IS_ERR_RAISE(status); } ret = PyString_FromStringAndSize((char *)blob.data, blob.length); talloc_free(mem_ctx); |