)]}'
{
  "commit": "4b9dd97dd7d2d8b0b34cfc9af446efa9b61118ee",
  "tree": "8f96f8bf07deefd9d7a0557a867f5b382d88e2c4",
  "parents": [
    "70ca6bc24be103dabd68e448cd3af29b929b771d"
  ],
  "author": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Wed Nov 29 16:38:01 2023 -0500"
  },
  "committer": {
    "name": "Boringssl LUCI CQ",
    "email": "boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Wed Nov 29 21:58:04 2023 +0000"
  },
  "message": "Add a value barrier in p224_select_point\n\nClang seems to be undoing the constant-time code here. This is another\nconstant-time table lookup, so this is more of the usual problem. Found\nwith the valgrind-based tooling.\n\nI didn\u0027t switch this to constant_time_conditional_memxor since we still\nhaven\u0027t figured out https://crbug.com/boringssl/655, but we definitely\nneed some kind of abstraction for this pattern.\n\nChange-Id: Ic11873b23cde31375ac1a326ed09ac1ca53ec913\nReviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/64310\nReviewed-by: Bob Beck \u003cbbe@google.com\u003e\nCommit-Queue: David Benjamin \u003cdavidben@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "9d0242c68c15d07aa99816532ea8a3e64edb355d",
      "old_mode": 33188,
      "old_path": "crypto/fipsmodule/ec/p224-64.c",
      "new_id": "d2b7f9f38108885b04e335ca11a1ead09237f99a",
      "new_mode": 33188,
      "new_path": "crypto/fipsmodule/ec/p224-64.c"
    }
  ]
}
