)]}'
{
  "commit": "6bd1e1504670dc96a76eb9858da4117bba586a41",
  "tree": "71e07239dbbaa9f2df7955f7b93c41ac28097219",
  "parents": [
    "df13691019be62da728e0c63463fe51bb5d7c29d"
  ],
  "author": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Wed Jul 19 13:56:50 2023 -0400"
  },
  "committer": {
    "name": "Boringssl LUCI CQ",
    "email": "boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Wed Jul 19 18:47:48 2023 +0000"
  },
  "message": "Silence -Wformat-truncation warning in crypto/err/err.c\n\nThis warning was being tripped because lib_buf and reason_buf made GCC,\nincorrectly, believe that the strings could get that long, and then\nattempted to sum up the snprintf to 120, obtained by inlining some\nthings.\n\nThose buffers were larger than they needed to be, so bringing it down is\nsufficient to silence things. That said, the buffer bounds are supplied\nby the caller and it is expected that truncation can occur, so the\nwarning is just incorrect. The warning can also be silenced by checking\nthe snprintf return value. As we\u0027re already trying to detect truncation,\nwe may as well do it with the return value and skip the extra strlen\ncall.\n\nEither of the two changes is sufficient to suppress the warning, but\nboth seem worthwhile, so I\u0027ve done them both.\n\nChange-Id: Ia1b1de67bba55da6f0d07e3682165a1820ce2c9e\nReviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/61805\nAuto-Submit: David Benjamin \u003cdavidben@google.com\u003e\nReviewed-by: Bob Beck \u003cbbe@google.com\u003e\nCommit-Queue: Bob Beck \u003cbbe@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "d041c5fec832a8b83eb545250ed93fa3407d4fc9",
      "old_mode": 33188,
      "old_path": "crypto/err/err.c",
      "new_id": "e8cf11408b6a5ff58168e642b18bcff183e1fe2c",
      "new_mode": 33188,
      "new_path": "crypto/err/err.c"
    }
  ]
}
