[ovs-git] [openvswitch/ovs] f5a12b: ovsdb-idl: Improve comments.

GitHub noreply at github.com
Mon Dec 11 22:31:23 UTC 2017


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: f5a12b82b82c1533dc5673680af87ad6cf270baf
      https://github.com/openvswitch/ovs/commit/f5a12b82b82c1533dc5673680af87ad6cf270baf
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-12-11 (Mon, 11 Dec 2017)

  Changed paths:
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-idl: Improve comments.

This change documents the IDL state machine, adds other comments,
and fixes a spelling error in a comment.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>


  Commit: 1ac62a0e09c018b584fbc1882dec6efc58fd40c2
      https://github.com/openvswitch/ovs/commit/1ac62a0e09c018b584fbc1882dec6efc58fd40c2
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-12-11 (Mon, 11 Dec 2017)

  Changed paths:
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-idl: Fix indentation in a couple of places.

White space changes only.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>


  Commit: fca1b42abe3eb6057d522694b80e92d70b3e51cc
      https://github.com/openvswitch/ovs/commit/fca1b42abe3eb6057d522694b80e92d70b3e51cc
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-12-11 (Mon, 11 Dec 2017)

  Changed paths:
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-idl: Fix assertion failure on error path parsing server reply.

If the database server sent an error reply to a monitor_cond request, and
the error was not a JSON string, then passing the error to json_string()
caused an assertion failure.

Found by inspection.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/a81da0805742...fca1b42abe3e


More information about the git mailing list