Bug 1817768 - Filter out illegal file extensions. r=Gijs

Differential Revision: https://phabricator.services.mozilla.com/D170531
mozilla · Feb 27, 2023 · 4cf1876 · 4cf1876
1 parent 7e59313
commit 4cf1876
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/widget/windows/nsFilePicker.cpp b/widget/windows/nsFilePicker.cpp
@@ -271,7 +271,11 @@ bool nsFilePicker::ShowFilePicker(const nsString& aInitialDir) {
 
   // default extension to append to new files
   if (!mDefaultExtension.IsEmpty()) {
-    hr = dialog->SetDefaultExtension(mDefaultExtension.get());
+    // We don't want environment variables expanded in the extension either.
+    nsAutoString sanitizedExtension(mDefaultExtension);
+    sanitizedExtension.ReplaceChar('%', '_');
+
+    hr = dialog->SetDefaultExtension(sanitizedExtension.get());
     if (FAILED(hr)) {
       return false;
     }