Fix dangling assignment of relationpath for Postgres 18.#13
Open
robert-pang wants to merge 1 commit into
Open
Conversation
…alue of GetRelationPath() is an rvalue for the duration of the assignment only. Signed-off-by: Robert Pang <robertpang@google.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When compiling pgfincore with Postgres 18, we observed the following compiler warning:
From what we notice,
GetRelationPath()in Postgres 18 returns the relation path as the following struct as an rvalue that lasts for the duration of the assignment only:To keep a reference to the
strsafely, we propose a fix to duplicate it usingpstrdup()to resolve the compiler warning.