(WIP) Only free the topic filter string if the current topic owns it

awslabs / aws-crt-nodejs

NodeJS bindings for the AWS Common Runtime.

Apache License 2.0

37 stars 24 forks source link

(WIP) Only free the topic filter string if the current topic owns it #461

Closed TwistedTwigleg closed 1 year ago

TwistedTwigleg commented 1 year ago

Description of changes:

Changes to the topic tree to prevent segfaults when freeing the topic filter. Still WIP

This PR is just for more easily checking CI and building with the change - it is not intended to be merged into aws-crt-nodejs at this time nor with these changes like presented in this PR.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

TwistedTwigleg commented 1 year ago

Closing now that a solution was found in https://github.com/awslabs/aws-c-mqtt/pull/297