Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

Gss memory management fixups #152

Merged
merged 2 commits into from
Oct 31, 2023
Merged

Gss memory management fixups #152

merged 2 commits into from
Oct 31, 2023

Conversation

alonbl
Copy link
Contributor

@alonbl alonbl commented Oct 31, 2023

Two commits:

  • Fix for uGnssMsgPrivateReceiveStart: do not double free pInstance->pTemporaryBuffer
  • Fix for uGnssPosGetStreamedStart: do not release NULL mutex

@alonbl
Fix for uGnssMsgPrivateReceiveStart: do not double free pInstance->pT…
fd0a520 
…emporaryBuffer

Free at:
* uGnssMsgPrivateReceiveStart
* deleteGnssInstance
@alonbl
Fix for uGnssPosGetStreamedStart: do not release NULL mutex
8945c82 
Signed-off-by: Alon Bar-Lev <[email protected]>
@RobMeades
Copy link
Contributor

RobMeades commented Oct 31, 2023 •
edited
Loading

Many thanks and nicely found, will run through the test system...

@RobMeades RobMeades self-requested a review October 31, 2023 16:09
RobMeades
RobMeades approved these changes Oct 31, 2023
View reviewed changes
Copy link
Contributor

@RobMeades RobMeades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed that this passes all GNSS tests, thanks again.

@RobMeades RobMeades merged commit 09ce57b into u-blox:master Oct 31, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@RobMeades RobMeades RobMeades approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alonbl @RobMeades