Skip to content

krb5: use dynbuf - #13568

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/krb5-dynbuf
Closed

krb5: use dynbuf#13568
bagder wants to merge 1 commit into
masterfrom
bagder/krb5-dynbuf

Conversation

@bagder

@bagder bagder commented May 8, 2024

Copy link
Copy Markdown
Member

We lack tests for krb5 for FTP and I can't run any manual tests. This edit was done "in the blind" but I think it is generally a good idea even if there is a risk that I broke something here.

  • saves having to do repeated reallocs by reusing the dynbuf
  • avoids direct uses of realloc + memcpy

@bagder bagder closed this in 0f4c439 May 9, 2024
@bagder
bagder deleted the bagder/krb5-dynbuf branch May 9, 2024 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant