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.

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

Sanity check params memory usage #932

Open
tablatronix opened this issue Sep 10, 2019 · 2 comments
Open

Sanity check params memory usage #932

tablatronix opened this issue Sep 10, 2019 · 2 comments
Labels
DEV Help Wanted Developer Needs Help enhancement Feature Request Good First Issue PR for newbs
Milestone
dev

Comments

@tablatronix
Copy link
Collaborator

Since params are dynamic memory now, prevent overflow by checking memory before setting.
@tablatronix tablatronix added the enhancement Feature Request label Sep 10, 2019
@tablatronix tablatronix added this to the dev milestone Sep 10, 2019
@tablatronix tablatronix added DEV Help Wanted Developer Needs Help Good First Issue PR for newbs labels Sep 11, 2019
@naftalimurgor
Copy link

Hello @tablatronix
This seems interesting, what are useful pointers to get started with this, with regards to:

  • Setting things up
  • Testing - Unit testing, testing manually, etc.
  • Testing on an actual device

Thanks,
best,
Naftali

@debasis9658
Copy link

Hey @tablatronix I would like take on this issue. C an you assign if no one is working on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DEV Help Wanted Developer Needs Help enhancement Feature Request Good First Issue PR for newbs
Projects
None yet
Milestone
dev
Development

No branches or pull requests
3 participants
@tablatronix @naftalimurgor @debasis9658