TALLOC_FREE happily lives with a NULL ptr. Tim, please check!

[ira/wip.git] / source3 / smbd / nttrans.c

diff --git a/source3/smbd/nttrans.c b/source3/smbd/nttrans.c
index d51c9a6d67ea1863edc7084a69c2e76938e1508d..204cdf9e3109b79b3d1540d15066dd3254718164 100644 (file)
--- a/source3/smbd/nttrans.c
+++ b/source3/smbd/nttrans.c
@@ -1316,12 +1316,8 @@ static NTSTATUS copy_internals(TALLOC_CTX *ctx,
                goto out;
        }
  out:
-       if (smb_fname) {
-               TALLOC_FREE(smb_fname);
-       }
-       if (smb_fname_new) {
-               TALLOC_FREE(smb_fname_new);
-       }
+       TALLOC_FREE(smb_fname);
+       TALLOC_FREE(smb_fname_new);
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(3,("copy_internals: Error %s copy file %s to %s\n",
                        nt_errstr(status), oldname, newname));