Skip to content

process_common: fixed handle leak - #1237

Open
ProjectMutilation wants to merge 1 commit into
awslabs:mainfrom
ProjectMutilation:fix-handle-leak
Open

process_common: fixed handle leak#1237
ProjectMutilation wants to merge 1 commit into
awslabs:mainfrom
ProjectMutilation:fix-handle-leak

Conversation

@ProjectMutilation

Copy link
Copy Markdown
Contributor

When going on the on_finish label after check aws_byte_buf_append_dynamic(&result_buffer, &cursor), it is possible that the output_stream will not be closed.

Description of changes:

  • add implicit init of output_stream
  • replace closing in on_finish label
    By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Comment thread source/process_common.c Outdated
Signed-off-by: Maksim Korotkov <m.korotkov@postgrespro.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants