Skip to content
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.

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

LGTM.com - false positive - PyGame Surface (Call to function lambda with too many arguments; should be no more than 0.) #9426

Open
eforgacs opened this issue Jun 2, 2022 · 2 comments

Comments

@eforgacs
Copy link

@eforgacs eforgacs commented Jun 2, 2022

pygame.Surface() takes a Tuple[int, int] representing the width and height of a surface, according to its own documentation, located here: pygame.Surface — pygame v2.1.1 documentation

The program runs as intended with these parameters.

URL to the alert on the project page on LGTM.com:
https://lgtm.com/projects/g/eforgacs-games/Dragon_Warrior/snapshot/1eeb90fbf09ea8997b24acec694fa7f87a097755/files/data/text/dialog.py?sort=name&dir=ASC&mode=heatmap#x122458cab8b662ed:1

@tausbn
Copy link
Contributor

@tausbn tausbn commented Jun 3, 2022

Thank you for your report!

My best guess here is that we're somehow not picking up the signature of pygame.Surface correctly (which, as far as I can tell, is implemented in C).

It may also be something else that is confusing the analysis (since it refers to the imported Surface function as "function lambda", which doesn't make much sense).

Since this is not a security-related question, and as we're currently focusing on improving our security analysis, we'll add this to the backlog and deal with it in due course.

@eforgacs
Copy link
Author

@eforgacs eforgacs commented Jun 3, 2022

Sounds great! Thank you :)

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

No branches or pull requests

2 participants