From: Matt Corallo <git@bluematt.me>
Date: Mon, 27 Sep 2021 23:24:43 +0000 (+0000)
Subject: Correct clone logic for `Str`.
X-Git-Tag: v0.0.101.3^0
X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=dde6026d418c42ca5f0030a856e1ec15a2aa4d55;hp=dde6026d418c42ca5f0030a856e1ec15a2aa4d55;p=ldk-c-bindings

Correct clone logic for `Str`.

Previously we'd blindly clone'd the fields, which, if
`data_is_owned` is set, will always result in a a double-free.

Instead, we always clone the underlying bytes, setting
`data_is_owned` on the returned value since its likely the caller
wants to hold onto the string outside of the current context.
---