| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CAOcJUbyb0LgdNeCaJTuq+BHQqqPOXrtH9me7-KvtG+2BciUuxA@mail.gmail.com>
Date: Mon, 2 Oct 2017 06:34:31 -0400
From: Michael Ira Krufky <mkrufky@...uxtv.org>
To: Jérémy Lefaure <jeremy.lefaure@....epita.fr>
Cc: Hans Verkuil <hverkuil@...all.nl>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Sergey Kozlov <serjk@...up.ru>, Abylay Ospan <aospan@...up.ru>,
linux-media <linux-media@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 03/18] media: use ARRAY_SIZE
On Sun, Oct 1, 2017 at 3:30 PM, Jérémy Lefaure
<jeremy.lefaure@....epita.fr> wrote:
> Using the ARRAY_SIZE macro improves the readability of the code. Also,
> it is not always useful to use a variable to store this constant
> calculated at compile time.
>
> Found with Coccinelle with the following semantic patch:
> @r depends on (org || report)@
> type T;
> T[] E;
> position p;
> @@
> (
> (sizeof(E)@p /sizeof(*E))
> |
> (sizeof(E)@p /sizeof(E[...]))
> |
> (sizeof(E)@p /sizeof(T))
> )
>
> Signed-off-by: Jérémy Lefaure <jeremy.lefaure@....epita.fr>
Reviewed-by: Michael Ira Krufky <mkrufky@...uxtv.org>
> ---
> drivers/media/common/saa7146/saa7146_video.c | 9 ++++-----
> drivers/media/dvb-frontends/cxd2841er.c | 7 +++----
> drivers/media/pci/saa7146/hexium_gemini.c | 3 ++-
> drivers/media/pci/saa7146/hexium_orion.c | 3 ++-
> drivers/media/pci/saa7146/mxb.c | 3 ++-
> drivers/media/usb/dvb-usb/cxusb.c | 3 ++-
> drivers/media/usb/dvb-usb/friio-fe.c | 5 ++---
> 7 files changed, 17 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/media/common/saa7146/saa7146_video.c b/drivers/media/common/saa7146/saa7146_video.c
> index 37b4654dc21c..612aefd804f0 100644
> --- a/drivers/media/common/saa7146/saa7146_video.c
> +++ b/drivers/media/common/saa7146/saa7146_video.c
> @@ -4,6 +4,7 @@
> #include <media/v4l2-event.h>
> #include <media/v4l2-ctrls.h>
> #include <linux/module.h>
> +#include <linux/kernel.h>
>
> static int max_memory = 32;
>
> @@ -86,13 +87,11 @@ static struct saa7146_format formats[] = {
> due to this, it's impossible to provide additional *packed* formats, which are simply byte swapped
> (like V4L2_PIX_FMT_YUYV) ... 8-( */
>
> -static int NUM_FORMATS = sizeof(formats)/sizeof(struct saa7146_format);
> -
> struct saa7146_format* saa7146_format_by_fourcc(struct saa7146_dev *dev, int fourcc)
> {
> - int i, j = NUM_FORMATS;
> + int i;
>
> - for (i = 0; i < j; i++) {
> + for (i = 0; i < ARRAY_SIZE(formats); i++) {
> if (formats[i].pixelformat == fourcc) {
> return formats+i;
> }
> @@ -524,7 +523,7 @@ static int vidioc_s_fbuf(struct file *file, void *fh, const struct v4l2_framebuf
>
> static int vidioc_enum_fmt_vid_cap(struct file *file, void *fh, struct v4l2_fmtdesc *f)
> {
> - if (f->index >= NUM_FORMATS)
> + if (f->index >= ARRAY_SIZE(formats))
> return -EINVAL;
> strlcpy((char *)f->description, formats[f->index].name,
> sizeof(f->description));
> diff --git a/drivers/media/dvb-frontends/cxd2841er.c b/drivers/media/dvb-frontends/cxd2841er.c
> index 48ee9bc00c06..2cb97a3130be 100644
> --- a/drivers/media/dvb-frontends/cxd2841er.c
> +++ b/drivers/media/dvb-frontends/cxd2841er.c
> @@ -29,6 +29,7 @@
> #include <linux/math64.h>
> #include <linux/log2.h>
> #include <linux/dynamic_debug.h>
> +#include <linux/kernel.h>
>
> #include "dvb_math.h"
> #include "dvb_frontend.h"
> @@ -1696,12 +1697,10 @@ static u32 cxd2841er_dvbs_read_snr(struct cxd2841er_priv *priv,
> min_index = 0;
> if (delsys == SYS_DVBS) {
> cn_data = s_cn_data;
> - max_index = sizeof(s_cn_data) /
> - sizeof(s_cn_data[0]) - 1;
> + max_index = ARRAY_SIZE(s_cn_data) - 1;
> } else {
> cn_data = s2_cn_data;
> - max_index = sizeof(s2_cn_data) /
> - sizeof(s2_cn_data[0]) - 1;
> + max_index = ARRAY_SIZE(s2_cn_data) - 1;
> }
> if (value >= cn_data[min_index].value) {
> res = cn_data[min_index].cnr_x1000;
> diff --git a/drivers/media/pci/saa7146/hexium_gemini.c b/drivers/media/pci/saa7146/hexium_gemini.c
> index d31a2d4494d1..39357eddee32 100644
> --- a/drivers/media/pci/saa7146/hexium_gemini.c
> +++ b/drivers/media/pci/saa7146/hexium_gemini.c
> @@ -27,6 +27,7 @@
>
> #include <media/drv-intf/saa7146_vv.h>
> #include <linux/module.h>
> +#include <linux/kernel.h>
>
> static int debug;
> module_param(debug, int, 0);
> @@ -388,7 +389,7 @@ static struct saa7146_ext_vv vv_data = {
> .inputs = HEXIUM_INPUTS,
> .capabilities = 0,
> .stds = &hexium_standards[0],
> - .num_stds = sizeof(hexium_standards) / sizeof(struct saa7146_standard),
> + .num_stds = ARRAY_SIZE(hexium_standards),
> .std_callback = &std_callback,
> };
>
> diff --git a/drivers/media/pci/saa7146/hexium_orion.c b/drivers/media/pci/saa7146/hexium_orion.c
> index 043318aa19e2..461e421080f3 100644
> --- a/drivers/media/pci/saa7146/hexium_orion.c
> +++ b/drivers/media/pci/saa7146/hexium_orion.c
> @@ -27,6 +27,7 @@
>
> #include <media/drv-intf/saa7146_vv.h>
> #include <linux/module.h>
> +#include <linux/kernel.h>
>
> static int debug;
> module_param(debug, int, 0);
> @@ -460,7 +461,7 @@ static struct saa7146_ext_vv vv_data = {
> .inputs = HEXIUM_INPUTS,
> .capabilities = 0,
> .stds = &hexium_standards[0],
> - .num_stds = sizeof(hexium_standards) / sizeof(struct saa7146_standard),
> + .num_stds = ARRAY_SIZE(hexium_standards),
> .std_callback = &std_callback,
> };
>
> diff --git a/drivers/media/pci/saa7146/mxb.c b/drivers/media/pci/saa7146/mxb.c
> index 930218cc2de1..0144f305ea24 100644
> --- a/drivers/media/pci/saa7146/mxb.c
> +++ b/drivers/media/pci/saa7146/mxb.c
> @@ -30,6 +30,7 @@
> #include <media/v4l2-common.h>
> #include <media/i2c/saa7115.h>
> #include <linux/module.h>
> +#include <linux/kernel.h>
>
> #include "tea6415c.h"
> #include "tea6420.h"
> @@ -837,7 +838,7 @@ static struct saa7146_ext_vv vv_data = {
> .inputs = MXB_INPUTS,
> .capabilities = V4L2_CAP_TUNER | V4L2_CAP_VBI_CAPTURE | V4L2_CAP_AUDIO,
> .stds = &standard[0],
> - .num_stds = sizeof(standard)/sizeof(struct saa7146_standard),
> + .num_stds = ARRAY_SIZE(standard),
> .std_callback = &std_callback,
> };
>
> diff --git a/drivers/media/usb/dvb-usb/cxusb.c b/drivers/media/usb/dvb-usb/cxusb.c
> index 37dea0adc695..9b486bb5004d 100644
> --- a/drivers/media/usb/dvb-usb/cxusb.c
> +++ b/drivers/media/usb/dvb-usb/cxusb.c
> @@ -26,6 +26,7 @@
> #include <media/tuner.h>
> #include <linux/vmalloc.h>
> #include <linux/slab.h>
> +#include <linux/kernel.h>
>
> #include "cxusb.h"
>
> @@ -303,7 +304,7 @@ static int cxusb_aver_power_ctrl(struct dvb_usb_device *d, int onoff)
> 0x0e, 0x2, 0x47, 0x88,
> };
> msleep(20);
> - for (i = 0; i < sizeof(bufs)/sizeof(u8); i += 4/sizeof(u8)) {
> + for (i = 0; i < ARRAY_SIZE(bufs); i += 4 / sizeof(u8)) {
> ret = cxusb_ctrl_msg(d, CMD_I2C_WRITE,
> bufs+i, 4, &buf, 1);
> if (ret)
> diff --git a/drivers/media/usb/dvb-usb/friio-fe.c b/drivers/media/usb/dvb-usb/friio-fe.c
> index 0251a4e91d47..a6c84a4390d1 100644
> --- a/drivers/media/usb/dvb-usb/friio-fe.c
> +++ b/drivers/media/usb/dvb-usb/friio-fe.c
> @@ -13,6 +13,7 @@
> #include <linux/init.h>
> #include <linux/string.h>
> #include <linux/slab.h>
> +#include <linux/kernel.h>
>
> #include "friio.h"
>
> @@ -362,8 +363,6 @@ static u8 init_code[][2] = {
> {0x76, 0x0C},
> };
>
> -static const int init_code_len = sizeof(init_code) / sizeof(u8[2]);
> -
> static int jdvbt90502_init(struct dvb_frontend *fe)
> {
> int i = -1;
> @@ -377,7 +376,7 @@ static int jdvbt90502_init(struct dvb_frontend *fe)
> msg.addr = state->config.demod_address;
> msg.flags = 0;
> msg.len = 2;
> - for (i = 0; i < init_code_len; i++) {
> + for (i = 0; i < ARRAY_SIZE(init_code); i++) {
> msg.buf = init_code[i];
> ret = i2c_transfer(state->i2c, &msg, 1);
> if (ret != 1)
> --
> 2.14.1
>
Powered by blists - more mailing lists