Menu

#185 LUA signfor not functional

2.8.4
closed-fixed
None
6
2014-03-16
2013-12-23
No

odkim.signfor isn't working. It always reports:

odkim.signfor(): incorrect argument count

Looking into the code it looks like noone is calling
top = lua_gettop(l);
at the beginning.

Discussion

  • Murray S. Kucherawy

    I believe you're right. Patch applied for 2.9.1. Does that one-line change fix it for you?

     
  • Murray S. Kucherawy

    • assigned_to: Murray S. Kucherawy
    • Priority: 5 --> 6
     
  • Daniele Orlandi

    Daniele Orlandi - 2013-12-26

    Yes, the fix is okay.

    It is however of little use as signfor needs the SigningTable and if the SinignTable is present there is no way (AFAIK) to prevent signing if the policy implemented in LUA determines that it shouldn't.

    I'll open a new ticket for this.

     
  • Murray S. Kucherawy

    v2.9.1 released.

     
  • Murray S. Kucherawy

    • status: open --> closed-fixed
     

Log in to post a comment.