diff options
author | wm4 <wm4@nowhere> | 2014-08-11 13:00:57 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2014-08-11 13:00:57 +0200 |
commit | b016f92309403fbd3598e70699464692de8e8f4b (patch) | |
tree | 3d7adcfde3de9720c797d1905f6e43774a56c18f /player | |
parent | ece61bfa100ae4af032bea6ce9ffb6da86126525 (diff) |
command: fix dangling pointer issue in script key bindings code
This code was sending a string to a different thread, and then
deallocated the string shortly after, which means most of the time
the other thread was accessing a dangling pointer.
It's possible that this is the cause for #1002.
Diffstat (limited to 'player')
-rw-r--r-- | player/command.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/player/command.c b/player/command.c index 270798d925..cae7e02805 100644 --- a/player/command.c +++ b/player/command.c @@ -3894,7 +3894,7 @@ int run_command(MPContext *mpctx, mp_cmd_t *cmd) *event = (mpv_event_client_message){0}; for (int n = 1; n < cmd->nargs; n++) { MP_TARRAY_APPEND(event, event->args, event->num_args, - cmd->args[n].v.s); + talloc_strdup(event, cmd->args[n].v.s)); } if (mp_client_send_event(mpctx, cmd->args[0].v.s, MPV_EVENT_CLIENT_MESSAGE, event) < 0) |