[lvc-project] [PATCH] media: staging/intel-ipu3: css: Change mismatch pointer.

[Fedor Pchelkin]

lvc-project случайно выпал из копии письма.

On Wed, 27. May 14:37, Fedor Pchelkin wrote:
> On Mon, 25. May 15:45, Markov Gleb wrote:
> > From: Gleb Markov <markov.gi at npc-ksb.ru>
> > 
> > Checking p_tr in the case of working with p_pl is
> > perceived as a typo, since the identical check is
> > present for p_tr in the case of working with p_tr.
> > 
> > Change p_tr on p_pl.
> > 
> > Found by Linux Verification Center (linuxtesting.org) with Syzkaller.
> > 
> > Fixes: e11110a5b744 ("media: staging/intel-ipu3: css: Compute and program ccs")
> > Signed-off-by: Gleb Markov <markov.gi at npc-ksb.ru>
> > ---
> >  drivers/staging/media/ipu3/ipu3-css-params.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/staging/media/ipu3/ipu3-css-params.c b/drivers/staging/media/ipu3/ipu3-css-params.c
> > index 76ad802d694e..8cfe89c7f7ef 100644
> > --- a/drivers/staging/media/ipu3/ipu3-css-params.c
> > +++ b/drivers/staging/media/ipu3/ipu3-css-params.c
> > @@ -1617,7 +1617,7 @@ imgu_css_acc_process_lines(const struct process_lines *pl,
> >  				/* Inside the grid */
> >  				p_pl[pl_idx].lines = process_lines;
> >  
> > -			if (p_tr) {
> > +			if (p_pl) {
> 
> Мм... теперь Свейс на это будет ругаться как NULL_AFTER_DEREF.  Ведь во
> всех случаях на пару строк до проверки указатель p_pl спокойно
> разыменовывается.
> 
> Вроде бы здесь нет никакой опечатки?  Возможно с cfg_set нужно работать
> только когда p_tr ненулевой согласно логике драйвера, имеет смысл
> дополнительно попробовать изучить.
> 
> >  				p_pl[pl_idx].cfg_set = pl_cfg_set;
> >  				pl_cfg_set = 1 - pl_cfg_set;
> >  			}
> > -- 
> > 2.43.0