[dovecot/pigeonhole] 8b9f5e: lib-sieve: imap4flags extension: Fix binary corrup...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/dovecot/pigeonhole
  Commit: 8b9f5e2ed8df09b099c8dda57702a220e7487052
      https://github.com/dovecot/pigeonhole/commit/8b9f5e2ed8df09b099c8dda57702a220e7487052
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2018-01-04 (Thu, 04 Jan 2018)

  Changed paths:
    M src/lib-sieve/plugins/imap4flags/cmd-flag.c
    M src/lib-sieve/plugins/imap4flags/ext-imap4flags.c

  Log Message:
  -----------
  lib-sieve: imap4flags extension: Fix binary corruption occurring when setflag/addflag/removeflag flag-list is a variable.

The original implementation checked the first operand for being a variable.
Obviously, when the assigned flag string is just a variable, this doesn't work.
This causes the flag list to be recognized erroneously as a variable flags
string. Fixed the problem by explicitly emitting an omitted operand when there
is no second argument for a setflag/addflag/removeflag.

Because the new byte code format is incompatible, the extension version is
bumped to 1.


  Commit: d4ee6c857918d0e40f07706c3592ef092c9e296a
      https://github.com/dovecot/pigeonhole/commit/d4ee6c857918d0e40f07706c3592ef092c9e296a
  Author: Stephan Bosch <stephan.bosch at dovecot.fi>
  Date:   2018-01-04 (Thu, 04 Jan 2018)

  Changed paths:
    M tests/extensions/imap4flags/basic.svtest

  Log Message:
  -----------
  tests: imap4flags extension: Added more tests involving variables.


Compare: https://github.com/dovecot/pigeonhole/compare/00d2c5b75a03...d4ee6c857918