Changeset 32682 in vbox

Timestamp:

Sep 22, 2010 6:59:46 AM (14 years ago)

Author:

vboxsync

Message:

Main: fixed incorrect parameters checking in MediumFormat implementation

File:

• trunk/src/VBox/Main/MediumFormatImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/MediumFormatImpl.cpp

@@ -198 +198 @@
 STDMETHODIMP MediumFormat::COMGETTER(FileExtensions)(ComSafeArrayOut(BSTR, aFileExtensions))
 {
-    if (ComSafeArrayOutIsNull(aFileExtensions))
-        return E_POINTER;
+    CheckComArgOutSafeArrayPointerValid(aFileExtensions);
 
     AutoCaller autoCaller(this);
@@ -242 +241 @@
                                               ComSafeArrayOut(BSTR, aDefaults))
 {
-    CheckComArgSafeArrayNotNull(aNames);
-    CheckComArgSafeArrayNotNull(aDescriptions);
-    CheckComArgSafeArrayNotNull(aTypes);
-    CheckComArgSafeArrayNotNull(aFlags);
-    CheckComArgSafeArrayNotNull(aDefaults);
+    CheckComArgOutSafeArrayPointerValid(aNames);
+    CheckComArgOutSafeArrayPointerValid(aDescriptions);
+    CheckComArgOutSafeArrayPointerValid(aTypes);
+    CheckComArgOutSafeArrayPointerValid(aFlags);
+    CheckComArgOutSafeArrayPointerValid(aDefaults);
 
     AutoCaller autoCaller(this);