Fix mkdir, chdir, getattr and setattr

1 files changed, 7 insertions, 2 deletions

diff --git a/sftsr.c b/sftsr.c
index c510aae..cfab025 100644
--- a/sftsr.c
+++ b/sftsr.c
@@ -880,6 +880,7 @@ static void handle_getattr(union INTPACK __far *r)
 	}
 
 	map_shfl_info_to_getattr(r, &parms.create.Info);
+	(void)vbox_shfl_close(&data.vb, data.hgcm_client_id, root, parms.create.Handle);
 	clear_dos_err(r);
 }
 
@@ -888,7 +889,7 @@ static void handle_setattr(union INTPACK __far *r)
 	// TODO: Just silently ignoring setattr,
 	// since the only attribute we could try to set in a multiplatform way
 	// is A_RDONLY by messing with UNIX perms.
-	clear_dos_err(&r);
+	clear_dos_err(r);
 }
 
 /** Opens directory corresponding to file in path (i.e. we use the dirname),
@@ -1221,10 +1222,13 @@ static void handle_chdir(union INTPACK __far *r)
 	// Also check whether it is really a directory
 	if (!(map_shfl_attr_to_dosattr(&parms.create.Info.Attr) & _A_SUBDIR)) {
 		set_dos_err(r, DOS_ERROR_PATH_NOT_FOUND);
-		return;
+		goto chdir_close;
 	}
 
 	clear_dos_err(r);
+
+chdir_close:
+	(void)vbox_shfl_close(&data.vb, data.hgcm_client_id, root, parms.create.Handle);
 }
 
 static void handle_mkdir(union INTPACK __far *r)
@@ -1262,6 +1266,7 @@ static void handle_mkdir(union INTPACK __far *r)
 		break;
 	}
 
+	(void)vbox_shfl_close(&data.vb, data.hgcm_client_id, root, parms.create.Handle);
 	clear_dos_err(r);
 }