New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: handled empty scanner and export false case for vscan #3502

Open

Hardikl wants to merge 1 commit into main from hl_vscan

+9 −1

Contributor

Hardikl commented Mar 3, 2025

This change would be handling 2 cases,

When aggregation is enabled(which is default), Making each instance to false in 2nd loop instead of 1st loop.
When scanner value is empty, don't proceed and jump to next iteration for both the loops. -> Earlier no instances were created for this kind of bad data in 1st loop and 2nd loop would fail when no instance has been found.


          fix: handled empty scanner and export false case for vscan

ff17aa6

Hardikl requested review from cgrinds and rahulguptajss as code owners

March 3, 2025 09:42

cla-bot bot added the cla-signed label

Hardikl linked an issue

that may be closed by this pull request

Vscan counters are missing #3500

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels