AdguardBrowserExtension icon indicating copy to clipboard operation
AdguardBrowserExtension copied to clipboard

Memory Leak

Open JohnSheppard11 opened this issue 1 year ago • 2 comments

Please answer the following questions for yourself before submitting an issue

  • [X] Filters were updated before reproducing an issue
  • [X] I checked the knowledge base and found no answer
  • [X] I checked to make sure that this issue has not already been filed

AdGuard Extension version

5.0.113

Browser version

Google Chrome 129.0.6668.60

OS version

windows 10 22H2 19045.4894

Ad Blocking

No response

Privacy

No response

Social

No response

Annoyances

No response

Security

No response

Other

No response

Language-specific

No response

What Tracking protection options do you have enabled?

No response

Issue Details

Steps to reproduce:

  1. Launch

5 browser windows, ~30-50 tabs each. PC Power - sleep only.

1+ week already...

Temp fix: Reopen tab, for some sites after reopening you need to close the remaining processes/scripts through the browser task manager.

Expected Behavior

Working properly.

Actual Behavior

Memory leak happens. Why plz.

Screenshots

Screenshot 1

Additional Information

This bug may also be present in other browsers, check.

(Memory leak in 2024 imagine)

Using since 2011.

JohnSheppard11 avatar Sep 30 '24 04:09 JohnSheppard11

Yes, it is present in other browsers: Slimjet, for example. That's an outdated browser to be sure, but at least it points away from Chrome. This never happened with the MV2 version.

rpodric avatar Oct 04 '24 06:10 rpodric

This has been present for the past several versions and still persists for 5.0.138. A web page/tab that is open for a significant period of time and/or one that causes a lot of AdGuard extension blocking will cause an out of memory error. I've seen it in Edge, Brave and Vivaldi. I've had to stop using the extension and revert to uBOL in the meantime.

jimsinenomine avatar Oct 17 '24 06:10 jimsinenomine

Still present in 5.0.161

jimsinenomine avatar Oct 26 '24 13:10 jimsinenomine

I guess no one has looked at it yet. Maybe this is the wrong way to report.

rpodric avatar Oct 26 '24 14:10 rpodric

@jimsinenomine @JohnSheppard11 @rpodric Please reproduce the problem and open Task Manager in Chrome (Shift + Esc). How much memory does AdGuard consume and what is the CPU load? Can you reproduce the problem consistently on certain websites or can it happen on any website?

A web page/tab that is open for a significant period of time and/or one that causes a lot of AdGuard extension blocking will cause an out of memory error.

How long does the page have to be open? Do you just leave the page open or refresh it or click something on the page? Do you have any memory-saving options enabled in your browser?

It would also be great if you attach your AdGuard extension settings in the comment.

windwak3r avatar Oct 28 '24 10:10 windwak3r

The ones it's happened on for me are "voice.google.com" and "mail.google.com" (as PWA). I have them permanently open in Edge. Settings are attached. I've turned the extension back on and it should cause the error again (although I did notice an update to 5.0.162). Assuming it errors, I'll then do the task manager checks and report back.

I just leave the page open. No refresh (auto or manual). Memory setting in Edge is the default "Balanced savings".

adg_ext_settings_5.0.162_301024-125258.json

jimsinenomine avatar Oct 30 '24 18:10 jimsinenomine

In my case, it must have been YouTube, if the site matters at all, which I'm guessing that it doesn't, since I don't use that browser for anything else. I think it's much more the time on a given site rather than the site.

rpodric avatar Oct 30 '24 18:10 rpodric

Here you go re the error and the task manager. Not sure how much it helps. There's a theme here - in case anyone didn't notice 🙂 i.e. Google-owned web sites (Voice, Gmail, YouTube, etc.). Took 10-12 hours for this error to show up. HTH.

Edge OoM error Edge OoM - task manager

jimsinenomine avatar Oct 31 '24 23:10 jimsinenomine

Issue remains in 5.0.164

jimsinenomine avatar Nov 07 '24 05:11 jimsinenomine

I know there's a bit of a coincidence since all the sites mentioned so far relate to Google, so it may be instructive to try to replicate on some unrelated site, like Wikipedia or whatever else.

rpodric avatar Nov 07 '24 05:11 rpodric

Can confirm the same behavior on Windows 11 with GoogleChrome 130.0.6723.92 and AddGuard Extension 5.0.164. The problem occurs after a few hours of inactivity on various websites such as: E.g. support.sap.com and others.

The tabs appear blank with an “Out of Memory” error message and a high memory usage of 1GB to 4GB. System memory of 40 GB RAM almost completely used. Closing tabs frees up storage space accordingly. Disabling the AddGuard extension will completely resolve the issue.

scviktor avatar Nov 07 '24 12:11 scviktor

I don’t see the point in dumping anything, all open sites from the very opening start to slowly gain size in RAM, after 5 hours several GB are accumulated.

I don’t know how and where to open the settings, and I, with 5 open windows and 200 tabs, have no desire to risk losing data and turn on extension again.

I'm too lazy to log in to GitHub again and write something, I can't provide data from the logs and extension settings due to the large number of open tabs. So please don't expect any answers from me from now on. I provided everything I could provide.

So, to begin with, I recommend you to perform a global rollback to a stable version, then look for the causes of the bug, and redo all the tests properly, so as not to lose your audience and rating, which will be quite difficult to get back.

JohnSheppard11 avatar Nov 10 '24 22:11 JohnSheppard11

Not fixed in 5.0.167

jimsinenomine avatar Nov 16 '24 13:11 jimsinenomine

We are preparing a possible fix for this problem. Further discussions will take place at https://github.com/AdguardTeam/AdguardBrowserExtension/issues/2594

windwak3r avatar Nov 18 '24 17:11 windwak3r

Isn't that thread, a year old, about the MV2 version though? I never saw the problem with MV2.

rpodric avatar Nov 18 '24 17:11 rpodric

We are preparing a possible fix for this problem. Further discussions will take place at #2594

This version is only for Chrome not for Firefox :-(

Tasiorsa avatar Nov 25 '24 14:11 Tasiorsa

There will also be a fix for Firefox. It is currently under review.

maximtop avatar Nov 25 '24 15:11 maximtop

There will also be a fix for Firefox. It is currently under review.

But for firefox still 4.41 version - not the latest 4.48

Tasiorsa avatar Nov 25 '24 15:11 Tasiorsa

But for firefox still 4.41 version - not the latest 4.48

The Firefox version of AdGuard AdBlocker (4.4.48) is currently under review in the Firefox Add-ons store.

maximtop avatar Nov 25 '24 15:11 maximtop

Firefox 132.0.2 - Adguard 4.48 and after 48h still the same problem {8BE2A037-EA4C-4A69-ABB3-73B44766227C}

Tasiorsa avatar Nov 26 '24 08:11 Tasiorsa