)]}'
{
  "commit": "f1d153dc3619eaea7fac176716c53824f5584830",
  "tree": "617000509e9bbbf46e44087ae09f26bd8e5a7fb1",
  "parents": [
    "5799ebfe5f895ecd29fcd699789854fc1790d6cd"
  ],
  "author": {
    "name": "Adam Langley",
    "email": "alangley@gmail.com",
    "time": "Fri Jul 30 14:51:22 2021 -0700"
  },
  "committer": {
    "name": "Adam Langley",
    "email": "alangley@gmail.com",
    "time": "Mon Aug 02 17:10:02 2021 +0000"
  },
  "message": "Don\u0027t overread in poly_Rq_mul\n\nThe polynomials have 701, 16-bit values. But poly_Rq_mul was reading 32\nbytes at offset 1384 in order to get the last 18 of them. This silently\nworked for a long time, but when 7153013019 switched to keeping\nvariables on the stack it was noticed by Valgrind.\n\nThis change fixes the overread. Setting watchpoints at the ends of the\ntwo inputs (and one output) now shows no overreads nor overwrites.\n\nBUG\u003d424\n\nChange-Id: Id86c1407ffce66593541c10feee47213f4b95c5d\nReviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/48645\nReviewed-by: David Benjamin \u003cdavidben@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "53ce47ca817249c5a20b3d36824ad4b42b98182f",
      "old_mode": 33188,
      "old_path": "crypto/hrss/asm/poly_rq_mul.S",
      "new_id": "c37d7d0b4d361d711cd0ba5d63c8f8f0e215cee0",
      "new_mode": 33188,
      "new_path": "crypto/hrss/asm/poly_rq_mul.S"
    }
  ]
}
