[driver] cosmetic changes

none (or all) functions should be static. making all function non-static.

consistent format of function args (ie each argument on its own line).

adding __notnull for our fcb, fobx, vnetroot, netroot context pointers.

removed some non-implemented function from our redirector functions table.
left some no-op functions that are just required: flush, isvaliddir,
deallocatefcb. not sure about completebufferingstate function.

80char lines corrections

sys/nfs41_debug.c

@@ -618,8 +618,19 @@ const char *opcode2string(int opcode)
     }
 }
 
+void print_acl_args(
+    SECURITY_INFORMATION info)
+{
+    DbgP("Security query: %s %s %s\n",
+        (info & OWNER_SECURITY_INFORMATION)?"OWNER":"",
+        (info & GROUP_SECURITY_INFORMATION)?"GROUP":"",
+        (info & DACL_SECURITY_INFORMATION)?"DACL":"",
+        (info & SACL_SECURITY_INFORMATION)?"SACL":"");
+}
+
 void print_open_error(int on, int status)
 {
+    if (!on) return;
     switch (status) {
     case STATUS_NETWORK_ACCESS_DENIED:
         DbgP("[ERROR] nfs41_Create: STATUS_NETWORK_ACCESS_DENIED\n");

sys/nfs41_debug.h

@@ -57,6 +57,7 @@ const char *opcode2string(int opcode);
 void print_open_error(int on, int status);
 void print_wait_status(int on, const char *str, NTSTATUS status, 
                        const char *opcode, PVOID entry, int xid);
+void print_acl_args(SECURITY_INFORMATION info);
 
 #define DbgEn() DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_ERROR_LEVEL, \
         "--> [%s] [%d] %s\n", _DRIVER_NAME_, IoGetCurrentProcess(), \