nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1507055 - AntiTrackingCommon::RedirectHeuristic must check the number of redirect entries, r=Ehsan

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D60505

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Andrea Marchesini 2020-01-21 17:59:33 +00:00
parent f97dcc5818
commit b9bec0df94
1 changed files with 1 additions and 2 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

3
toolkit/components/antitracking/AntiTrackingCommon.cpp
View file

@ -2372,10 +2372,9 @@ void AntiTrackingCommon::RedirectHeuristic(nsIChannel* aOldChannel,
nsCOMPtr<nsIPrincipal> trackingPrincipal; nsCOMPtr<nsIPrincipal> trackingPrincipal;
nsCOMPtr<nsIURI> trackingURI; nsCOMPtr<nsIURI> trackingURI;
if (allowedByPreviousRedirect) { if (allowedByPreviousRedirect && !chain.IsEmpty()) {
const nsTArray<nsCOMPtr<nsIRedirectHistoryEntry>>& chain = const nsTArray<nsCOMPtr<nsIRedirectHistoryEntry>>& chain =
oldLoadInfo->RedirectChain(); oldLoadInfo->RedirectChain();
MOZ_ASSERT(chain.Length() > 0);
rv = chain[0]->GetPrincipal(getter_AddRefs(trackingPrincipal)); rv = chain[0]->GetPrincipal(getter_AddRefs(trackingPrincipal));
if (NS_WARN_IF(NS_FAILED(rv))) { if (NS_WARN_IF(NS_FAILED(rv))) {
LOG(("Can't obtain the principal from the redirect chain")); LOG(("Can't obtain the principal from the redirect chain"));