r4830: Fix for problem noticed by Guy Harris <gharris@apple.com>, return
authorJeremy Allison <jra@samba.org>
Tue, 18 Jan 2005 22:40:49 +0000 (22:40 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 15:53:57 +0000 (10:53 -0500)
correct DOS/NT error code on transact named pipe on closed pipe
handle.
Jeremy.
(This used to be commit 599c281464fa96725c3ee6dd3c5ee03ea81314ea)

source3/smbd/ipc.c

index 9fcd39b5002caa932a0ebbcd3163ec055fa6d42f..3f21a2ac6adcdf1ff8eb072009f951575a9d3bc9 100644 (file)
@@ -269,7 +269,7 @@ static int api_fd_reply(connection_struct *conn,uint16 vuid,char *outbuf,
        /* First find out the name of this file. */
        if (suwcnt != 2) {
                DEBUG(0,("Unexpected named pipe transaction.\n"));
-               return(-1);
+               return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
        }
 
        /* Get the file handle and hence the file name. */
@@ -290,7 +290,7 @@ static int api_fd_reply(connection_struct *conn,uint16 vuid,char *outbuf,
                }
 
                DEBUG(1,("api_fd_reply: INVALID PIPE HANDLE: %x\n", pnum));
-               return api_no_reply(outbuf, mdrcnt);
+               return ERROR_NT(NT_STATUS_INVALID_HANDLE);
        }
 
        DEBUG(3,("Got API command 0x%x on pipe \"%s\" (pnum %x)\n", subcommand, p->name, pnum));
@@ -315,6 +315,8 @@ static int api_fd_reply(connection_struct *conn,uint16 vuid,char *outbuf,
                /* Set Named Pipe Handle state */
                reply = api_SNPHS(outbuf, p, params, tpscnt);
                break;
+       default:
+               return ERROR_NT(NT_STATUS_INVALID_PARAMETER);
        }
 
        if (!reply)