From 4a9a569600cbcddffcfbeca97587de516259e939 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Wed, 24 Dec 2008 13:44:44 +0100 Subject: Use parent_dirname_talloc instead of parent_dirname in copy_internals --- source3/smbd/nttrans.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'source3') diff --git a/source3/smbd/nttrans.c b/source3/smbd/nttrans.c index 24a14a8c1b..c9874f6a48 100644 --- a/source3/smbd/nttrans.c +++ b/source3/smbd/nttrans.c @@ -1127,6 +1127,7 @@ static NTSTATUS copy_internals(TALLOC_CTX *ctx, int info; SMB_OFF_T ret=-1; NTSTATUS status = NT_STATUS_OK; + char *parent; ZERO_STRUCT(sbuf1); ZERO_STRUCT(sbuf2); @@ -1255,8 +1256,11 @@ static NTSTATUS copy_internals(TALLOC_CTX *ctx, /* Grrr. We have to do this as open_file_ntcreate adds aARCH when it creates the file. This isn't the correct thing to do in the copy case. JRA */ - file_set_dosmode(conn, newname, fattr, &sbuf2, - parent_dirname(newname),false); + if (!parent_dirname_talloc(talloc_tos(), newname, &parent, NULL)) { + return NT_STATUS_NO_MEMORY; + } + file_set_dosmode(conn, newname, fattr, &sbuf2, parent, false); + TALLOC_FREE(parent); if (ret < (SMB_OFF_T)sbuf1.st_size) { return NT_STATUS_DISK_FULL; -- cgit