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

[Security] UAF and SEGV on NanoMQ 0.22.4 #1861

Closed
roman-kraus-fokus opened this issue Aug 2, 2024 · 3 comments
Closed

[Security] UAF and SEGV on NanoMQ 0.22.4 #1861

roman-kraus-fokus opened this issue Aug 2, 2024 · 3 comments
Labels
Investigation quiestion remains ambiguous

Comments

@roman-kraus-fokus
Copy link

Describe the bug
We have found a heap-use-after-free on NanoMQ 0.22.4. The input we use for that sometimes triggers a segmentation fault instead.

To Reproduce
We have a proof of concept available. As it endangers the security of NanoMQ, we would prefer sharing it privately with you. Do you have a channel for us to do so?

Environment Details

  • NanoMQ version 0.22.4
  • Operating system: Ubuntu 20.04
  • Compiler: Clang 11
@JaylinYu
Copy link
Member

JaylinYu commented Aug 4, 2024

There is a known bug in newly added prefix/suffix feature, I guess that's what you just hit.

You can share any security issue Via Our slack channel or PM me there.

Join me on Slack -- it’s a faster, simpler way to work. Sign up here, from any device: https://join.slack.com/t/emqx/shared_invite/zt-2ntpicu1z-i1eBr6T7WwdMAD62TxqYwA

@JaylinYu JaylinYu added the Investigation quiestion remains ambiguous label Aug 4, 2024
@JaylinYu JaylinYu changed the title Heap-Use-After-Free and SEGV on NanoMQ 0.22.4 [Security] UAF and SEGV on NanoMQ 0.22.4 Aug 4, 2024
@JaylinYu
Copy link
Member

JaylinYu commented Aug 5, 2024

let's continue session on slack. or plz post your poc data here.
thx

@JaylinYu JaylinYu closed this as completed Aug 5, 2024
@roman-kraus-fokus
Copy link
Author

Sure, we made some final checks and will contact you shortly on Slack.

JaylinYu added a commit to nanomq/NanoNNG that referenced this issue Aug 12, 2024
two bugs hidden under it. uncared proto_Data in retain msg
another one is still suspicous, remoing pipe from a lock protected sendq caused SEGV.

Signed-off-by: jaylin <jaylin@emqx.io>
JaylinYu added a commit to nanomq/NanoNNG that referenced this issue Aug 12, 2024
two bugs hidden under it. uncared proto_Data in retain msg
another one is still suspicous, remoing pipe from a lock protected sendq caused SEGV.

Signed-off-by: jaylin <jaylin@emqx.io>
JaylinYu added a commit that referenced this issue Aug 12, 2024
Signed-off-by: jaylin <jaylin@emqx.io>
JaylinYu added a commit that referenced this issue Aug 12, 2024
Signed-off-by: jaylin <jaylin@emqx.io>
JaylinYu added a commit that referenced this issue Aug 12, 2024
Signed-off-by: jaylin <jaylin@emqx.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Investigation quiestion remains ambiguous
Projects
None yet
Development

No branches or pull requests

2 participants