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

find_ty_cmdid panics if the buffer is too small. #113

Open
todo bot opened this issue Dec 22, 2018 · 0 comments
Open

find_ty_cmdid panics if the buffer is too small. #113

todo bot opened this issue Dec 22, 2018 · 0 comments
Assignees
@roblabla
Labels
project-libuser Related to the libuser todo 🗒️ Created by the todo bot type-bug Something isn't working
Milestone
KFS-7

Comments

@todo
Copy link

todo bot commented Dec 22, 2018

We should return an error if the buf size is too small instead of panicking

https://github.com/roblabla42/KFS/blob/adee47274ba5c0dc0ef112ce8933becd63d3da77/libuser/src/ipc/mod.rs#L799-L807

This issue was generated by todo based on a TODO comment in adee472 when #90 was merged. cc @roblabla.
@todo todo bot assigned roblabla Dec 22, 2018
@todo todo bot added the todo 🗒️ Created by the todo bot label Dec 22, 2018
@roblabla roblabla added type-bug Something isn't working project-libuser Related to the libuser labels Dec 23, 2018
@roblabla roblabla added this to the kfs-7 milestone Jan 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roblabla roblabla

Labels
project-libuser Related to the libuser todo 🗒️ Created by the todo bot type-bug Something isn't working
Projects
None yet
Milestone
KFS-7
Development

No branches or pull requests
1 participant
@roblabla