)]}'
{
  "commit": "08921ac9e9d185d6f472238ea9c79a3a70eb7ea7",
  "tree": "2cdc6ad9191e6b601ef3fc5eedb67ec305cbbee0",
  "parents": [
    "792bc09a9896dfbdac575ab4cdcd0f4db96d8ca1"
  ],
  "author": {
    "name": "Jesper Juhl",
    "email": "jj@chaosbits.net",
    "time": "Thu Apr 21 18:11:25 2011 -0300"
  },
  "committer": {
    "name": "Mauro Carvalho Chehab",
    "email": "mchehab@redhat.com",
    "time": "Fri May 20 09:28:57 2011 -0300"
  },
  "message": "[media] Media, DVB, Siano, smsusb: Avoid static analysis report about \u0027use after free\u0027\n\nIn drivers/media/dvb/siano/smsusb.c we have this code:\n ...\n               kfree(dev);\n\n               sms_info(\"device %p destroyed\", dev);\n ...\n\nat least one static analysis tool (Coverity Prevent) complains about this\nas a use-after-free bug.\nWhile it\u0027s true that we do use the pointer variable after freeing it, the\nonly use is to print the value of the pointer, so there\u0027s not actually any\nproblem here. But still, silencing the complaint is trivial by just moving\nthe kfree() call below the sms_info(), so why not just do it?. It doesn\u0027t\nchange the workings of the code in any way, but it makes the tool shut up.\nThe patch below also removes a rather pointless blank line.\n\nSigned-off-by: Jesper Juhl \u003cjj@chaosbits.net\u003e\nSigned-off-by: Mauro Carvalho Chehab \u003cmchehab@redhat.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "0b8da57cf4c340fcd0336e5a65ee40f0b47fb8f4",
      "old_mode": 33188,
      "old_path": "drivers/media/dvb/siano/smsusb.c",
      "new_id": "0c8164a2cc36caf271c34848049290068b4ba855",
      "new_mode": 33188,
      "new_path": "drivers/media/dvb/siano/smsusb.c"
    }
  ]
}
