> Hello. > > On 03/06/2017 05:20 PM, Elena Reshetova wrote: > > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to use-after-free > > situations. > > > > Signed-off-by: Elena Reshetova <elena.reshet...@intel.com> > > Signed-off-by: Hans Liljestrand <ishkam...@gmail.com> > > Signed-off-by: Kees Cook <keesc...@chromium.org> > > Signed-off-by: David Windsor <dwind...@gmail.com> > [...] > > diff --git a/drivers/media/pci/cx88/cx88.h b/drivers/media/pci/cx88/cx88.h > > index 115414c..16c1313 100644 > > --- a/drivers/media/pci/cx88/cx88.h > > +++ b/drivers/media/pci/cx88/cx88.h > > @@ -24,6 +24,7 @@ > > #include <linux/i2c-algo-bit.h> > > #include <linux/videodev2.h> > > #include <linux/kdev_t.h> > > +#include <linux/refcount.h> > > > > #include <media/v4l2-device.h> > > #include <media/v4l2-fh.h> > > @@ -339,7 +340,7 @@ struct cx8802_dev; > > > > struct cx88_core { > > struct list_head devlist; > > - atomic_t refcount; > > + refcount_t refcount; > > Could you please keep the name aligned with above and below?
You mean "not aligned" to devlist, but with a shift like it was before? Sure, will fix. Is the patch ok otherwise? Best Regards, Elena. > > > > > /* board name */ > > int nr; > > > > MBR, Sergei -- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To unsubscribe from this group and stop receiving emails from it, send an email to open-iscsi+unsubscr...@googlegroups.com. To post to this group, send email to open-iscsi@googlegroups.com. Visit this group at https://groups.google.com/group/open-iscsi. For more options, visit https://groups.google.com/d/optout.