Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Lua: Fix stack level after register odkim functions. #201

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Lua: Fix stack level after register odkim functions. #201

wants to merge 1 commit into from

Commits on Feb 27, 2024

  1. Lua: Fix stack level after register odkim functions. 

    When luaL_setfuncs() is replaced by luaL_newlib() + lua_setglobal()
in commit 74b3374, the commit did
not take account that lua_setglobal() pops a value from the stack.

Thus, following lua pop(l, 1) tries to pop from empty stack in
Lua >= 5.2, especially in Lua 5.4, it causes abort().

This fix it.

* miltertest/miltertest.c (main):
  Don't pop after lua_setglobal() in Lua >= 5.2

* opendkim/opendkim-lua.c
  (dkimf_lua_setup_hook, dkimf_lua_screen_hook,
   dkimf_lua_stats_hook, dkimf_lua_final_hook): As above.
    @futatuki
    futatuki committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    57b86d3 View commit details
    Browse the repository at this point in the history