summaryrefslogtreecommitdiff
path: root/source4/ntvfs
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2009-07-15 19:17:57 +0200
committerStefan Metzmacher <metze@samba.org>2009-07-15 19:24:35 +0200
commit5240b10eaa1f19dcf2dbc31e5e1f8868716bbc69 (patch)
tree433a096029875bfb14e6192891e0fcdffbcf5c23 /source4/ntvfs
parent382d5c8f5bb3b54e7c1a2daaf9d0283b6275768a (diff)
downloadsamba-5240b10eaa1f19dcf2dbc31e5e1f8868716bbc69.tar.gz
samba-5240b10eaa1f19dcf2dbc31e5e1f8868716bbc69.tar.bz2
samba-5240b10eaa1f19dcf2dbc31e5e1f8868716bbc69.zip
s3:ntvfs/posix: avoid unnesessary talloc_reference()
This caused the panics on the RAW-SETFILEINFO.RENAME test, because we returned an empty strings. The problem was: ERROR: talloc_steal with references at ntvfs/posix/pvfs_setfileinfo.c:215 reference at ntvfs/posix/pvfs_resolve.c:799 metze
Diffstat (limited to 'source4/ntvfs')
-rw-r--r--source4/ntvfs/posix/pvfs_resolve.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/source4/ntvfs/posix/pvfs_resolve.c b/source4/ntvfs/posix/pvfs_resolve.c
index c33323350e..8e8da72e1f 100644
--- a/source4/ntvfs/posix/pvfs_resolve.c
+++ b/source4/ntvfs/posix/pvfs_resolve.c
@@ -796,7 +796,10 @@ NTSTATUS pvfs_resolve_parent(struct pvfs_state *pvfs, TALLOC_CTX *mem_ctx,
(*name)->has_wildcard = false;
/* we can't get the correct 'original_name', but for the purposes
of this call this is close enough */
- (*name)->original_name = talloc_reference(*name, child->original_name);
+ (*name)->original_name = talloc_strdup(*name, child->original_name);
+ if ((*name)->original_name == NULL) {
+ return NT_STATUS_NO_MEMORY;
+ }
(*name)->stream_name = NULL;
(*name)->stream_id = 0;