media: intel-ipu3: cio2: fixup off-by-one bug in cio2_vb2_buf_init
authorYong Zhi <yong.zhi@intel.com>
Fri, 19 Jan 2018 06:27:34 +0000 (01:27 -0500)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 23 Jan 2018 13:08:44 +0000 (08:08 -0500)
commit401f69308fe344b4353fad5233d99c7efa9deef5
treedd957532b69bf1863097e14c945e1a03e1273060
parent5eb8c768f49361dfd72ac55c82051695b7ed5ee3
media: intel-ipu3: cio2: fixup off-by-one bug in cio2_vb2_buf_init

With "pages" initialized to vb length + 1 pages, the condition
check if(!pages--) will break at one more page than intended,
this can result in out-of-bound access to b->lop[i][j] when setting
the last dummy page.

Fixes: c7cbef1fdb54 ("media: intel-ipu3: cio2: fix a crash with out-of-bounds access")
Signed-off-by: Yong Zhi <yong.zhi@intel.com>
Signed-off-by: Cao Bing Bu <bingbu.cao@intel.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/pci/intel/ipu3/ipu3-cio2.c