)]}'
{
  "commit": "0debbaa399bf60274a4382206e3d1aa96f63005c",
  "tree": "8b8a82b90a50e03673d78ecfd37a31caa4f0564a",
  "parents": [
    "e77b2589667c12afc7063f2f8b0c2c264ee665f6"
  ],
  "author": {
    "name": "David Barker",
    "email": "david.barker@argondesign.com",
    "time": "Fri Feb 23 14:39:01 2018 +0000"
  },
  "committer": {
    "name": "Tom Finegan",
    "email": "tomfinegan@google.com",
    "time": "Fri Feb 23 18:03:34 2018 +0000"
  },
  "message": "[NORMATIVE-SYNTAX, obu-sizing] Fix length field size handling\n\nIt is possible, and reasonable from an encoder perspective, to\nencode an OBU size field using more than the minimal number of\nbytes. With the current reference decoder, such lengths are\ndecoded correctly, but the data pointer is then advanced by the\nwrong number of bytes, leading to a desync.\n\nThis can easily be fixed by having aom_uleb_decode() return the\nexact number of bytes read. The behaviour of returning -1 on error\ncan be preserved.\n\nIn addition, fix aom_uleb_encode_fixed_size(), and strengthen its\ntests - this function appears to have been padding its output\nincorrectly, but this was not caught by the test suite.\n\nThe \u0027encode\u0027 tests also mixed big-endian and little-endian in a\npotentially confusing way (reading back the encoded bytes big-endian\nfor comparison to a 32-bit hex constant). Change to using an array\nof expected bytes, to make things clearer.\n\nBUG\u003daomedia:1433\n\nChange-Id: Iea80a1b70299e0a26cd45e44ecb039804f8c5245\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "10b70408e09f72e6d4935b218f1c8728c133ff3e",
      "old_mode": 33188,
      "old_path": "aom/aom_integer.h",
      "new_id": "6aa0c11de680e393cba2935043c0ba042342ad47",
      "new_mode": 33188,
      "new_path": "aom/aom_integer.h"
    },
    {
      "type": "modify",
      "old_id": "eb88220641570b8d4865e2ddc8d5ca3086204a51",
      "old_mode": 33188,
      "old_path": "aom/src/aom_integer.c",
      "new_id": "596953951b0b4e39c778ae9bd226bd04d2ca3183",
      "new_mode": 33188,
      "new_path": "aom/src/aom_integer.c"
    },
    {
      "type": "modify",
      "old_id": "b6394120be332d903bc9aac809b60bb775eeb1af",
      "old_mode": 33188,
      "old_path": "av1/decoder/obu.c",
      "new_id": "f5a194cfed843373a578e2a4cf126e5dfa0a902e",
      "new_mode": 33188,
      "new_path": "av1/decoder/obu.c"
    },
    {
      "type": "modify",
      "old_id": "ea184ceceb3b38a7753b01089f3da3de83b35d8c",
      "old_mode": 33188,
      "old_path": "obudec.c",
      "new_id": "a0cb85e1d4fb68918c48d76873ad7a36520b4b5f",
      "new_mode": 33188,
      "new_path": "obudec.c"
    },
    {
      "type": "modify",
      "old_id": "11263d53b073ee11713c0ba538062fcab00024c0",
      "old_mode": 33188,
      "old_path": "test/aom_integer_test.cc",
      "new_id": "48ceaffe7d80ba5bab59adc973b263b3d891a594",
      "new_mode": 33188,
      "new_path": "test/aom_integer_test.cc"
    },
    {
      "type": "modify",
      "old_id": "4c10455c30835baca446e028002403e87ff4f7fc",
      "old_mode": 33188,
      "old_path": "tools/obu_parser.cc",
      "new_id": "8a608c86cc4f6660085da63fb3c2f1d4e8166ad0",
      "new_mode": 33188,
      "new_path": "tools/obu_parser.cc"
    }
  ]
}
