Fix off-by-one error (bug 408431). r=dcamp, sr=jst, a=jst
git-svn-id: svn://10.0.0.236/trunk@241355 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
25
mozilla/dom/src/offline/crashtests/408431-1.html
Normal file
25
mozilla/dom/src/offline/crashtests/408431-1.html
Normal file
@@ -0,0 +1,25 @@
|
||||
<!DOCTYPE html>
|
||||
<html>
|
||||
<head>
|
||||
<script>
|
||||
|
||||
try {
|
||||
navigator.pendingOfflineLoads[0];
|
||||
} catch (e) {
|
||||
}
|
||||
|
||||
try {
|
||||
navigator.pendingOfflineLoads[1000];
|
||||
} catch (e) {
|
||||
}
|
||||
|
||||
try {
|
||||
navigator.pendingOfflineLoads[-1];
|
||||
} catch (e) {
|
||||
}
|
||||
|
||||
</script>
|
||||
</head>
|
||||
<body>
|
||||
</body>
|
||||
</html>
|
||||
1
mozilla/dom/src/offline/crashtests/crashtests.list
Normal file
1
mozilla/dom/src/offline/crashtests/crashtests.list
Normal file
@@ -0,0 +1 @@
|
||||
load 408431-1.html
|
||||
@@ -322,7 +322,7 @@ nsDOMOfflineLoadStatusList::Item(PRUint32 aItem, nsIDOMLoadStatus **aStatus)
|
||||
nsresult rv = Init();
|
||||
NS_ENSURE_SUCCESS(rv, rv);
|
||||
|
||||
if ((PRInt32)aItem > mItems.Count()) return NS_ERROR_DOM_INDEX_SIZE_ERR;
|
||||
if ((PRInt32)aItem >= mItems.Count()) return NS_ERROR_DOM_INDEX_SIZE_ERR;
|
||||
|
||||
NS_ADDREF(*aStatus = mItems[aItem]);
|
||||
|
||||
@@ -497,7 +497,6 @@ nsDOMOfflineLoadStatusList::Observe(nsISupports *aSubject,
|
||||
const char *aTopic,
|
||||
const PRUnichar *aData)
|
||||
{
|
||||
nsresult rv;
|
||||
if (!strcmp(aTopic, "offline-cache-update-added")) {
|
||||
nsCOMPtr<nsIOfflineCacheUpdate> update = do_QueryInterface(aSubject);
|
||||
if (update) {
|
||||
|
||||
@@ -13,6 +13,7 @@ include ../../content/xul/content/crashtests/crashtests.list
|
||||
include ../../docshell/base/crashtests/crashtests.list
|
||||
|
||||
include ../../dom/src/base/crashtests/crashtests.list
|
||||
include ../../dom/src/offline/crashtests/crashtests.list
|
||||
|
||||
include ../../editor/libeditor/html/crashtests/crashtests.list
|
||||
|
||||
|
||||
Reference in New Issue
Block a user