| 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: <DC6B3471-8721-49CA-B2E3-CA883E60D341@mellanox.com>
Date: Sun, 19 Feb 2017 17:21:58 +0000
From: Majd Dibbiny <majd@...lanox.com>
To: SF Markus Elfring <elfring@...rs.sourceforge.net>
CC: "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
Doug Ledford <dledford@...hat.com>,
Hal Rosenstock <hal.rosenstock@...il.com>,
"Leon Romanovsky" <leonro@...lanox.com>,
Matan Barak <matanb@...lanox.com>,
"Sean Hefty" <sean.hefty@...el.com>,
Yishai Hadas <yishaih@...lanox.com>,
LKML <linux-kernel@...r.kernel.org>,
"kernel-janitors@...r.kernel.org" <kernel-janitors@...r.kernel.org>
Subject: Re: [PATCH 10/29] IB/mlx4: Enclose 15 expressions for the sizeof
operator by parentheses
> On Feb 18, 2017, at 10:59 PM, SF Markus Elfring <elfring@...rs.sourceforge.net> wrote:
>
> From: Markus Elfring <elfring@...rs.sourceforge.net>
> Date: Sat, 18 Feb 2017 09:54:15 +0100
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> The script "checkpatch.pl" pointed information out like the following.
>
> WARNING: sizeof … should be sizeof(…)
>
> Thus fix the affected source code places.
>
> Signed-off-by: Markus Elfring <elfring@...rs.sourceforge.net>
> ---
> drivers/infiniband/hw/mlx4/mad.c | 30 +++++++++++++++---------------
> 1 file changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/infiniband/hw/mlx4/mad.c b/drivers/infiniband/hw/mlx4/mad.c
> index 860fec8b9601..b26817f0669f 100644
> --- a/drivers/infiniband/hw/mlx4/mad.c
> +++ b/drivers/infiniband/hw/mlx4/mad.c
> @@ -195,7 +195,7 @@ static void update_sm_ah(struct mlx4_ib_dev *dev, u8 port_num, u16 lid, u8 sl)
> if (!dev->send_agent[port_num - 1][0])
> return;
>
> - memset(&ah_attr, 0, sizeof ah_attr);
> + memset(&ah_attr, 0, sizeof(ah_attr));
> ah_attr.dlid = lid;
> ah_attr.sl = sl;
> ah_attr.port_num = port_num;
> @@ -400,7 +400,7 @@ static void forward_trap(struct mlx4_ib_dev *dev, u8 port_num, const struct ib_m
> * it's OK for our devices).
> */
> spin_lock_irqsave(&dev->sm_lock, flags);
> - memcpy(send_buf->mad, mad, sizeof *mad);
> + memcpy(send_buf->mad, mad, sizeof(*mad));
> send_buf->ah = dev->sm_ah[port_num - 1];
> if (send_buf->ah)
> ret = ib_post_send_mad(send_buf, NULL);
> @@ -555,7 +555,7 @@ int mlx4_ib_send_to_slave(struct mlx4_ib_dev *dev, int slave, u8 port,
>
> /* create ah. Just need an empty one with the port num for the post send.
> * The driver will set the force loopback bit in post_send */
> - memset(&attr, 0, sizeof attr);
> + memset(&attr, 0, sizeof(attr));
> attr.port_num = port;
> if (is_eth) {
> union ib_gid sgid;
> @@ -590,8 +590,8 @@ int mlx4_ib_send_to_slave(struct mlx4_ib_dev *dev, int slave, u8 port,
>
> /* copy over to tunnel buffer */
> if (grh)
> - memcpy(&tun_mad->grh, grh, sizeof *grh);
> - memcpy(&tun_mad->mad, mad, sizeof *mad);
> + memcpy(&tun_mad->grh, grh, sizeof(*grh));
> + memcpy(&tun_mad->mad, mad, sizeof(*mad));
>
> /* adjust tunnel data */
> tun_mad->hdr.pkey_index = cpu_to_be16(tun_pkey_ix);
> @@ -961,7 +961,7 @@ static int iboe_process_mad(struct ib_device *ibdev, int mad_flags, u8 port_num,
> }
> mutex_unlock(&dev->counters_table[port_num - 1].mutex);
> if (stats_avail) {
> - memset(out_mad->data, 0, sizeof out_mad->data);
> + memset(out_mad->data, 0, sizeof(out_mad->data));
> switch (counter_stats.counter_mode & 0xf) {
> case 0:
> edit_counter(&counter_stats,
> @@ -1136,11 +1136,11 @@ static void handle_slaves_guid_change(struct mlx4_ib_dev *dev, u8 port_num,
> if (!mlx4_is_mfunc(dev->dev) || !mlx4_is_master(dev->dev))
> return;
>
> - in_mad = kmalloc(sizeof *in_mad, GFP_KERNEL);
> + in_mad = kmalloc(sizeof(*in_mad), GFP_KERNEL);
> if (!in_mad)
> return;
>
> - out_mad = kmalloc(sizeof *out_mad, GFP_KERNEL);
> + out_mad = kmalloc(sizeof(*out_mad), GFP_KERNEL);
> if (!out_mad)
> goto free_in_mad;
>
> @@ -1149,8 +1149,8 @@ static void handle_slaves_guid_change(struct mlx4_ib_dev *dev, u8 port_num,
> for (i = 0; i < 4; i++) {
> if (change_bitmap && (!((change_bitmap >> (8 * i)) & 0xff)))
> continue;
> - memset(in_mad, 0, sizeof *in_mad);
> - memset(out_mad, 0, sizeof *out_mad);
> + memset(in_mad, 0, sizeof(*in_mad));
> + memset(out_mad, 0, sizeof(*out_mad));
>
> in_mad->base_version = 1;
> in_mad->mgmt_class = IB_MGMT_CLASS_SUBN_LID_ROUTED;
> @@ -1421,7 +1421,7 @@ int mlx4_ib_send_to_wire(struct mlx4_ib_dev *dev, int slave, u8 port,
> sizeof (struct mlx4_mad_snd_buf),
> DMA_TO_DEVICE);
>
> - memcpy(&sqp_mad->payload, mad, sizeof *mad);
> + memcpy(&sqp_mad->payload, mad, sizeof(*mad));
>
> ib_dma_sync_single_for_device(&dev->ib_dev,
> sqp->tx_ring[wire_tx_ix].buf.map,
> @@ -1804,7 +1804,7 @@ static int create_pv_sqp(struct mlx4_ib_demux_pv_ctx *ctx,
>
> tun_qp = &ctx->qp[qp_type];
>
> - memset(&qp_init_attr, 0, sizeof qp_init_attr);
> + memset(&qp_init_attr, 0, sizeof(qp_init_attr));
> qp_init_attr.init_attr.send_cq = ctx->cq;
> qp_init_attr.init_attr.recv_cq = ctx->cq;
> qp_init_attr.init_attr.sq_sig_type = IB_SIGNAL_ALL_WR;
> @@ -1837,7 +1837,7 @@ static int create_pv_sqp(struct mlx4_ib_demux_pv_ctx *ctx,
> return ret;
> }
>
> - memset(&attr, 0, sizeof attr);
> + memset(&attr, 0, sizeof(attr));
> attr.qp_state = IB_QPS_INIT;
> ret = 0;
> if (create_tun)
> @@ -2184,7 +2184,7 @@ static int mlx4_ib_alloc_demux_ctx(struct mlx4_ib_dev *dev,
> goto err_mcg;
> }
>
> - snprintf(name, sizeof name, "mlx4_ibt%d", port);
> + snprintf(name, sizeof(name), "mlx4_ibt%d", port);
> ctx->wq = alloc_ordered_workqueue(name, WQ_MEM_RECLAIM);
> if (!ctx->wq) {
> pr_err("Failed to create tunnelling WQ for port %d\n", port);
> @@ -2192,7 +2192,7 @@ static int mlx4_ib_alloc_demux_ctx(struct mlx4_ib_dev *dev,
> goto err_wq;
> }
>
> - snprintf(name, sizeof name, "mlx4_ibud%d", port);
> + snprintf(name, sizeof(name), "mlx4_ibud%d", port);
> ctx->ud_wq = alloc_ordered_workqueue(name, WQ_MEM_RECLAIM);
> if (!ctx->ud_wq) {
> pr_err("Failed to create up/down WQ for port %d\n", port);
> --
> 2.11.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Thanks,
Reviewed-by: Majd Dibbiny
<majd@...lanox.com>
Powered by blists - more mailing lists