From: Amine Khaldi <amine.khaldi@reactos.org>
Date: Tue, 12 May 2015 10:57:24 +0000 (+0000)
Subject: [SHELL32] Don't provide the format option when the volume is read only. By Barrett... 
X-Git-Tag: backups/colins-printing-for-freedom@73041~24^2~27
X-Git-Url: https://git.reactos.org/?p=reactos.git;a=commitdiff_plain;h=4f9b940d54d5fbe8ce347346f9492a7e4975d588

[SHELL32] Don't provide the format option when the volume is read only. By Barrett Karish. CORE-9530

svn path=/trunk/; revision=67683
---

diff --git a/reactos/dll/win32/shell32/CDefaultContextMenu.cpp b/reactos/dll/win32/shell32/CDefaultContextMenu.cpp
index 7a917ce5dd0..cdb97c1dd6e 100644
--- a/reactos/dll/win32/shell32/CDefaultContextMenu.cpp
+++ b/reactos/dll/win32/shell32/CDefaultContextMenu.cpp
@@ -14,6 +14,9 @@ TODO:
 
 #include "precomp.h"
 
+/* FIXME: ntifs.h */
+#define FILE_READ_ONLY_VOLUME 0x00080000
+
 WINE_DEFAULT_DEBUG_CHANNEL(dmenu);
 
 typedef struct _DynamicShellEntry_
@@ -880,11 +883,20 @@ CDefaultContextMenu::BuildShellItemContextMenu(
 
     if (_ILIsDrive(m_apidl[0]))
     {
-        /* The 'Format' option must be always available,
-         * thus it is not registered as a static shell extension */
-        _InsertMenuItemW(hMenu, IndexMenu++, TRUE, 0, MFT_SEPARATOR, NULL, 0);
-        _InsertMenuItemW(hMenu, IndexMenu++, TRUE, 0x7ABC, MFT_STRING, MAKEINTRESOURCEW(IDS_FORMATDRIVE), MFS_ENABLED);
-        bAddSep = TRUE;
+        char szDrive[8] = {0};
+        DWORD dwFlags;
+
+        _ILGetDrive(m_apidl[0], szDrive, sizeof(szDrive));
+        if (GetVolumeInformationA(szDrive, NULL, 0, NULL, NULL, &dwFlags, NULL, 0))
+        {
+            /* Disable format if read only */
+            if (!(dwFlags & FILE_READ_ONLY_VOLUME))
+            {
+                _InsertMenuItemW(hMenu, IndexMenu++, TRUE, 0, MFT_SEPARATOR, NULL, 0);
+                _InsertMenuItemW(hMenu, IndexMenu++, TRUE, 0x7ABC, MFT_STRING, MAKEINTRESOURCEW(IDS_FORMATDRIVE), MFS_ENABLED);
+                bAddSep = TRUE;
+            }
+        }
     }
 
     BOOL bClipboardData = (HasClipboardData() && (rfg & SFGAO_FILESYSTEM));