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

Memory issue causing crash when delete called on AMQPWorker* create via new #18

Open
mathemaphysics opened this issue Feb 11, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@mathemaphysics
Copy link
Owner

This happens in the unit tests and needs to be reconciled because it will likely become a problem later. When auto worker = new AMQPWorker(...) is called, it should later be possible to delete worker. This doesn't work correctly in the setup fixture class in tests/agent_t.cpp.

@mathemaphysics mathemaphysics changed the title Memory issue causing crazy when delete called on AMQPWorker* create via new Memory issue causing crash when delete called on AMQPWorker* create via new Feb 11, 2022
@mathemaphysics mathemaphysics self-assigned this Feb 11, 2022
@mathemaphysics mathemaphysics added the bug Something isn't working label Feb 11, 2022
@mathemaphysics
Copy link
Owner Author

It just occurred to me that we could declare these in-place using the constructor notation, but then I realized that this isn't possible because GoogleTest fixtures don't have constructors. Silly man.

@mathemaphysics mathemaphysics removed their assignment Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant