The Samba-Bugzilla – Attachment 10002 Details for
Bug 10646
crash open_file_ntcreate: Frame not freed in order
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Patch for 4.1 series
0001-Fix-several-talloc-stack-frames-not-freed.4.1.patch (text/plain), 3.01 KB, created by
Samuel Cabrero
on 2014-06-02 13:59:20 UTC
(
hide
)
Description:
Patch for 4.1 series
Filename:
MIME Type:
Creator:
Samuel Cabrero
Created:
2014-06-02 13:59:20 UTC
Size:
3.01 KB
patch
obsolete
>From 609ed3aebbca4736e9292c274d12cc65153ac8f2 Mon Sep 17 00:00:00 2001 >From: Samuel Cabrero <scabrero@zentyal.com> >Date: Wed, 28 May 2014 17:57:31 +0200 >Subject: [PATCH] Fix several talloc stack frames not freed > >Signed-off-by: Samuel Cabrero <scabrero@zentyal.com> >--- > source3/passdb/pdb_samba_dsdb.c | 8 ++++++++ > source4/torture/libnetapi/libnetapi.c | 1 + > 2 files changed, 9 insertions(+) > >diff --git a/source3/passdb/pdb_samba_dsdb.c b/source3/passdb/pdb_samba_dsdb.c >index 01e747a..febf21c 100644 >--- a/source3/passdb/pdb_samba_dsdb.c >+++ b/source3/passdb/pdb_samba_dsdb.c >@@ -339,6 +339,7 @@ static int pdb_samba_dsdb_replace_by_sam(struct pdb_samba_dsdb_state *state, > > msg = ldb_msg_new(frame); > if (!msg) { >+ talloc_free(frame); > return false; > } > >@@ -375,6 +376,7 @@ static int pdb_samba_dsdb_replace_by_sam(struct pdb_samba_dsdb_state *state, > pw, strlen(pw), > (void *)&pw_utf16.data, > &pw_utf16.length)) { >+ talloc_free(frame); > return LDB_ERR_OPERATIONS_ERROR; > } > ret |= ldb_msg_add_value(msg, "clearTextPassword", &pw_utf16, NULL); >@@ -1011,6 +1013,7 @@ static NTSTATUS pdb_samba_dsdb_getgrgid(struct pdb_methods *m, GROUP_MAP *map, > > status = idmap_xids_to_sids(state->idmap_ctx, tmp_ctx, id_maps); > if (!NT_STATUS_IS_OK(status)) { >+ talloc_free(tmp_ctx); > return status; > } > status = pdb_samba_dsdb_getgrsid(m, map, *id_map.sid); >@@ -1521,6 +1524,7 @@ static NTSTATUS pdb_samba_dsdb_delete_alias(struct pdb_methods *m, > > if (ldb_transaction_start(state->ldb) != LDB_SUCCESS) { > DEBUG(0, ("Failed to start transaction in dsdb_add_domain_alias(): %s\n", ldb_errstring(state->ldb))); >+ talloc_free(tmp_ctx); > return NT_STATUS_INTERNAL_ERROR; > } > >@@ -1542,15 +1546,18 @@ static NTSTATUS pdb_samba_dsdb_delete_alias(struct pdb_methods *m, > DEBUG(10, ("ldb_delete failed %s\n", > ldb_errstring(state->ldb))); > ldb_transaction_cancel(state->ldb); >+ talloc_free(tmp_ctx); > return NT_STATUS_LDAP(rc); > } > > if (ldb_transaction_commit(state->ldb) != LDB_SUCCESS) { > DEBUG(0, ("Failed to commit transaction in pdb_samba_dsdb_delete_alias(): %s\n", > ldb_errstring(state->ldb))); >+ talloc_free(tmp_ctx); > return NT_STATUS_INTERNAL_ERROR; > } > >+ talloc_free(tmp_ctx); > return NT_STATUS_OK; > } > >@@ -2061,6 +2068,7 @@ static bool pdb_samba_dsdb_gid_to_sid(struct pdb_methods *m, gid_t gid, > > status = idmap_xids_to_sids(state->idmap_ctx, tmp_ctx, id_maps); > if (!NT_STATUS_IS_OK(status)) { >+ talloc_free(tmp_ctx); > return false; > } > *sid = *id_map.sid; >diff --git a/source4/torture/libnetapi/libnetapi.c b/source4/torture/libnetapi/libnetapi.c >index c516976..431b531 100644 >--- a/source4/torture/libnetapi/libnetapi.c >+++ b/source4/torture/libnetapi/libnetapi.c >@@ -35,6 +35,7 @@ bool torture_libnetapi_init_context(struct torture_context *tctx, > > if (!lp_load(lpcfg_configfile(tctx->lp_ctx), true, false, false, true)) { > fprintf(stderr, "error loading %s\n", lpcfg_configfile(tctx->lp_ctx)); >+ talloc_free(frame); > return W_ERROR_V(WERR_GENERAL_FAILURE); > } > >-- >1.9.1 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Raw
Actions:
View
Attachments on
bug 10646
:
10001
|
10002
|
10006