[vim/vim] fix dangling cmdline pointer after skip_expr_cctx (PR #20148)

[mattn]

Extract the cmdline restoration logic from compile_lambda into a helper restore_cmdline_arg() and call it from skip_expr_cctx() too, so a skipped lambda inside an "else" branch does not leave "*arg" pointing into freed evalarg memory.

Fixes #20147

---

You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/20148

Commit Summary

• 60d4ece fix dangling cmdline pointer after skip_expr_cctx

File Changes

(2 files)

• M src/testdir/test_vim9_cmd.vim (18)
• M src/vim9expr.c (37)

Patch Links:

• https://github.com/vim/vim/pull/20148.patch
• https://github.com/vim/vim/pull/20148.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/20148@github.com>

[Foxe Chen]

64-bitman left a comment (vim/vim#20148)

This fixes the issue for me. Thanks

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/20148/c4390183916@github.com>

[Christian Brabandt]

chrisbra left a comment (vim/vim#20148)

thanks

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/20148/c4390753904@github.com>

[Christian Brabandt]

Closed #20148 via 4bcc8ba.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/20148/issue_event/25230746823@github.com>