Messages in this thread | | | Date | Mon, 22 Nov 2021 13:42:30 +0300 | From | Dan Carpenter <> | Subject | [ti:ti-rt-linux-5.10.y 7558/10206] drivers/media/mc/mc-entity.c:443 __media_pipeline_start() error: uninitialized symbol 'ret'. |
| |
tree: git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git ti-rt-linux-5.10.y head: 0c67d996db8f3c9149598bc98657ae28fee22208 commit: 80dac899053d08524a40f195905022bb08e6104c [7558/10206] media: entity: Move the pipeline from entity to pads config: x86_64-randconfig-m001-20211116 (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings: drivers/media/mc/mc-entity.c:443 __media_pipeline_start() error: uninitialized symbol 'ret'.
vim +/ret +443 drivers/media/mc/mc-entity.c
20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 406 __must_check int __media_pipeline_start(struct media_entity *entity, e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 407 struct media_pipeline *pipe) e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 408 { d10c98949d1a1f drivers/media/media-entity.c Javier Martinez Canillas 2015-08-19 409 struct media_device *mdev = entity->graph_obj.mdev; 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 410 struct media_graph *graph = &pipe->graph; cd448544d8c6d3 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 411 struct media_pad *pad = entity->pads; cd448544d8c6d3 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 412 struct media_pad *pad_err = pad; 57208e5e25f263 drivers/media/media-entity.c Mauro Carvalho Chehab 2015-08-07 413 struct media_link *link; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 414 int ret; ^^^^^^^
e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 415 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 416 if (!pipe->streaming_count++) { 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 417 ret = media_graph_walk_init(&pipe->graph, mdev); 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 418 if (ret) 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 419 goto error_graph_walk_start; 106b9907c368e3 drivers/media/media-entity.c Sakari Ailus 2015-12-16 420 } 106b9907c368e3 drivers/media/media-entity.c Sakari Ailus 2015-12-16 421 cd448544d8c6d3 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 422 media_graph_walk_start(&pipe->graph, pad); cd448544d8c6d3 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 423 cd448544d8c6d3 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 424 while ((pad = media_graph_walk_next(graph))) { cd448544d8c6d3 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 425 struct media_entity *entity = pad->entity; 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 426 bool skip_validation = pad->pipe != NULL; 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 427 struct media_pad *iter; e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 428 ef69ee1bc2c10f drivers/media/media-entity.c Mauro Carvalho Chehab 2015-10-01 429 DECLARE_BITMAP(active, MEDIA_ENTITY_MAX_PADS); ef69ee1bc2c10f drivers/media/media-entity.c Mauro Carvalho Chehab 2015-10-01 430 DECLARE_BITMAP(has_no_links, MEDIA_ENTITY_MAX_PADS); af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 431 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 432 media_entity_for_each_pad(entity, iter) { 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 433 if (iter->pipe && iter->pipe != pipe) { 3eb87773dee633 drivers/media/media-entity.c Shuah Khan 2019-02-22 434 pr_err("Pipe active for %s. Can't start for %s\n", 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 435 entity->name, iter->entity->name); 8aaf62b5b9bef7 drivers/media/media-entity.c Sakari Ailus 2015-11-29 436 ret = -EBUSY;
"ret" is not necessarily initialized on the success path.
80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 437 } else { 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 438 iter->pipe = pipe; 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 439 } 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 440 iter->stream_count++; 8aaf62b5b9bef7 drivers/media/media-entity.c Sakari Ailus 2015-11-29 441 } 8aaf62b5b9bef7 drivers/media/media-entity.c Sakari Ailus 2015-11-29 442 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 @443 if (ret) ^^^
80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 444 goto error; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 445 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 446 /* Already part of the pipeline, skip validation. */ 80dac899053d08 drivers/media/mc/mc-entity.c Sakari Ailus 2021-07-14 447 if (skip_validation) af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 448 continue; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 449
--- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
| |