[ovs-git] [ovn-org/ovn] 5478ca: ovn-northd: Fix router policy pkt mark over flow i...

[numansiddique]

  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: 5478caa48c9497f6603e04b1c09d3df1c2b64e01
      https://github.com/ovn-org/ovn/commit/5478caa48c9497f6603e04b1c09d3df1c2b64e01
  Author: Numan Siddique <numans at ovn.org>
  Date:   2020-09-21 (Mon, 21 Sep 2020)

  Changed paths:
    M lib/ovn-util.c
    M lib/ovn-util.h
    M northd/ovn-northd.c
    M tests/ovn.at

  Log Message:
  -----------
  ovn-northd: Fix router policy pkt mark over flow if the value is greater than signed int.

If the value of pkt_mark in the router policy options is greater than 2147483647, we
are ignoring it. This is because we use smap_get_int(). This patch fixes this issue
by adding a wrapper function - ovn_smap_get_uint(). The OpenvSwitch library doesn't have
this smap helper function.

ovn_smap_get_uint() needs to be removed once the OpenvSwitch library has this helper
function. A comment has been added in the code for this.

Fixes: a123ef0fb8fd("Support packet metadata marking for logical router policies.")
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1878248
Reported-by: Alexander Constantinescu <aconstan at redhat.com>
Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>