diff --git a/mozilla/widget/src/mac/nsDynamicMDEF.cpp b/mozilla/widget/src/mac/nsDynamicMDEF.cpp index 8f6f295a219..a2fd01dcaa9 100644 --- a/mozilla/widget/src/mac/nsDynamicMDEF.cpp +++ b/mozilla/widget/src/mac/nsDynamicMDEF.cpp @@ -100,7 +100,8 @@ nsMenuStack::~nsMenuStack() nsresult nsMenuStack::GetMenuAt(PRInt32 aIndex, nsIMenu **outMenu) { - nsCOMPtr elementPtr = getter_AddRefs(mMenuArray.ElementAt(aIndex)); + // I'd prefer to fix the caller, but this will be safe + nsCOMPtr elementPtr = getter_AddRefs(mMenuArray.SafeElementAt(aIndex)); if (!elementPtr) return NS_ERROR_NULL_POINTER; @@ -111,7 +112,8 @@ nsMenuStack::GetMenuAt(PRInt32 aIndex, nsIMenu **outMenu) PRBool nsMenuStack::HaveMenuAt(PRInt32 aIndex) { - nsCOMPtr elementPtr = getter_AddRefs(mMenuArray.ElementAt(aIndex)); + // I'd prefer to fix the caller, but this will be safe + nsCOMPtr elementPtr = getter_AddRefs(mMenuArray.SafeElementAt(aIndex)); if (!elementPtr) return NS_ERROR_NULL_POINTER;