This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
[PATCH 03/11] Add necessary braces to if statements
- From: Peter Foley <pefoley2 at pefoley dot com>
- To: cygwin-patches at cygwin dot com
- Cc: Peter Foley <pefoley2 at pefoley dot com>
- Date: Sat, 19 Mar 2016 13:45:49 -0400
- Subject: [PATCH 03/11] Add necessary braces to if statements
- Authentication-results: sourceware.org; auth=none
- References: <1458409557-13156-1-git-send-email-pefoley2 at pefoley dot com>
The missing braces cause only the first expression to be guarded by the
else clause.
winsup/cygwin/ChangeLog
* fhandler_disk_file.cc (facl): Add missing braces to if statement.
* mount.cc (dos_drive_mappings): Add missing braces to if statement.
Signed-off-by: Peter Foley <pefoley2@pefoley.com>
---
winsup/cygwin/fhandler_disk_file.cc | 3 ++-
winsup/cygwin/mount.cc | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/winsup/cygwin/fhandler_disk_file.cc b/winsup/cygwin/fhandler_disk_file.cc
index 470dae8..2cf738f 100644
--- a/winsup/cygwin/fhandler_disk_file.cc
+++ b/winsup/cygwin/fhandler_disk_file.cc
@@ -1055,11 +1055,12 @@ cant_access_acl:
case GETACL:
if (!aclbufp)
set_errno(EFAULT);
- else
+ else {
res = getacl (get_stat_handle (), pc, nentries, aclbufp);
/* For this ENOSYS case, see security.cc:get_file_attribute(). */
if (res == -1 && get_errno () == ENOSYS)
goto cant_access_acl;
+ }
break;
case GETACLCNT:
res = getacl (get_stat_handle (), pc, 0, NULL);
diff --git a/winsup/cygwin/mount.cc b/winsup/cygwin/mount.cc
index ece8745..22bc49c 100644
--- a/winsup/cygwin/mount.cc
+++ b/winsup/cygwin/mount.cc
@@ -1962,7 +1962,7 @@ dos_drive_mappings::dos_drive_mappings ()
HANDLE sh = FindFirstVolumeW (vol, 64);
if (sh == INVALID_HANDLE_VALUE)
debug_printf ("FindFirstVolumeW, %E");
- else
+ else {
do
{
/* Skip drives which are not mounted. */
@@ -2023,6 +2023,7 @@ dos_drive_mappings::dos_drive_mappings ()
}
while (FindNextVolumeW (sh, vol, 64));
FindVolumeClose (sh);
+ }
}
wchar_t *
--
2.7.4