)]}'
{
  "commit": "1a87f285b5898ab63f54ea7e9a4e521a22078032",
  "tree": "bb7305d022bae02f6bed3f7a4c9a7213536ba821",
  "parents": [
    "12391e648d3c6f04a718721568cc89208b780654"
  ],
  "author": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Mon Aug 12 13:40:07 2024 -0400"
  },
  "committer": {
    "name": "Boringssl LUCI CQ",
    "email": "boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Thu Aug 15 17:53:56 2024 +0000"
  },
  "message": "Comment the core BN_div loop\n\nWith reference to Knuth.\n\nI\u0027m not sure what the comment about overflowing when q \u003d 0 is about. The\nbounds in \"the first part of the loop\" imply that we\u0027ve either computed\nq+1 or q and the borrow check exactly captures the q+1 case.\n\nMoreover, this addition is expected to *always* overflow. It cancels out\nthe underflow from subtracting too many.\n\nBug: 358687140\nChange-Id: I24bf8c9c37dcd1145667d7f0e8457c0e63e8783c\nReviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/70178\nReviewed-by: Bob Beck \u003cbbe@google.com\u003e\nCommit-Queue: David Benjamin \u003cdavidben@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "a4c0ded9274c360d13ab612c31c4e6f71ddc7db3",
      "old_mode": 33188,
      "old_path": "crypto/fipsmodule/bn/div.c",
      "new_id": "5d059e5c4781c7be5d64c6c11262b4c593c1c21a",
      "new_mode": 33188,
      "new_path": "crypto/fipsmodule/bn/div.c"
    }
  ]
}
