Author: abartlet
Date: 2007-05-15 06:51:18 +0000 (Tue, 15 May 2007)
New Revision: 22882

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=22882

Log:
It seems entirly reasonable to follow metze's suggestion and check for
a valid file handle first.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/ntvfs/ipc/vfs_ipc.c


Changeset:
Modified: branches/SAMBA_4_0/source/ntvfs/ipc/vfs_ipc.c
===================================================================
--- branches/SAMBA_4_0/source/ntvfs/ipc/vfs_ipc.c       2007-05-15 05:42:16 UTC 
(rev 22881)
+++ branches/SAMBA_4_0/source/ntvfs/ipc/vfs_ipc.c       2007-05-15 06:51:18 UTC 
(rev 22882)
@@ -610,14 +610,13 @@
                              struct ntvfs_request *req, union smb_fileinfo 
*info)
 {
        struct ipc_private *private = ntvfs->private_data;
+       struct pipe_state *p = pipe_state_find(private, 
info->generic.in.file.ntvfs);
+       if (!p) {
+               return NT_STATUS_INVALID_HANDLE;
+       }
        switch (info->generic.level) {
        case RAW_FILEINFO_GENERIC: 
        {
-               struct pipe_state *p;
-               p = pipe_state_find(private, info->generic.in.file.ntvfs);
-               if (!p) {
-                       return NT_STATUS_INVALID_HANDLE;
-               }
                ZERO_STRUCT(info->generic.out);
                info->generic.out.attrib = FILE_ATTRIBUTE_NORMAL;
                info->generic.out.fname.s = strrchr(p->pipe_name, '\\');

Reply via email to