media: drivers: improve a size determination

Replace the specification of a data structure by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.

This issue was detected by using the Coccinelle software.

[mchehab@s-opensoure.com: merge similar patches into one]

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Hans Verkuil <hansverk@cisco.com>

diff --git a/drivers/media/common/cypress_firmware.c b/drivers/media/common/cypress_firmware.c
index bfe47bc5f71672e36e379c0df7eb68f7c6de42b9..8895158c1962c6acbba24d80a7ee85643a54c113 100644 (file)
--- a/drivers/media/common/cypress_firmware.c
+++ b/drivers/media/common/cypress_firmware.c
@@ -74,7 +74,7 @@ int cypress_load_firmware(struct usb_device *udev,
        struct hexline *hx;
        int ret, pos = 0;
 
-       hx = kmalloc(sizeof(struct hexline), GFP_KERNEL);
+       hx = kmalloc(sizeof(*hx), GFP_KERNEL);
        if (!hx)
                return -ENOMEM;
 

diff --git a/drivers/media/common/siano/smscoreapi.c b/drivers/media/common/siano/smscoreapi.c
index 889b486fbc721feac943e48f0b0a1218e70f3ba0..ad1c41f727b1f1b4cd6d1b72201f164bc4db3344 100644 (file)
--- a/drivers/media/common/siano/smscoreapi.c
+++ b/drivers/media/common/siano/smscoreapi.c
@@ -447,7 +447,7 @@ static struct smscore_registry_entry_t *smscore_find_registry(char *devpath)
                        return entry;
                }
        }
-       entry = kmalloc(sizeof(struct smscore_registry_entry_t), GFP_KERNEL);
+       entry = kmalloc(sizeof(*entry), GFP_KERNEL);
        if (entry) {
                entry->mode = default_mode;
                strcpy(entry->devpath, devpath);
@@ -536,9 +536,7 @@ int smscore_register_hotplug(hotplug_t hotplug)
        int rc = 0;
 
        kmutex_lock(&g_smscore_deviceslock);
-
-       notifyee = kmalloc(sizeof(struct smscore_device_notifyee_t),
-                          GFP_KERNEL);
+       notifyee = kmalloc(sizeof(*notifyee), GFP_KERNEL);
        if (notifyee) {
                /* now notify callback about existing devices */
                first = &g_smscore_devices;
@@ -627,7 +625,7 @@ smscore_buffer_t *smscore_createbuffer(u8 *buffer, void *common_buffer,
 {
        struct smscore_buffer_t *cb;
 
-       cb = kzalloc(sizeof(struct smscore_buffer_t), GFP_KERNEL);
+       cb = kzalloc(sizeof(*cb), GFP_KERNEL);
        if (!cb)
                return NULL;
 
@@ -655,7 +653,7 @@ int smscore_register_device(struct smsdevice_params_t *params,
        struct smscore_device_t *dev;
        u8 *buffer;
 
-       dev = kzalloc(sizeof(struct smscore_device_t), GFP_KERNEL);
+       dev = kzalloc(sizeof(*dev), GFP_KERNEL);
        if (!dev)
                return -ENOMEM;
 
@@ -1684,7 +1682,7 @@ static int smscore_validate_client(struct smscore_device_t *coredev,
                pr_err("The msg ID already registered to another client.\n");
                return -EEXIST;
        }
-       listentry = kzalloc(sizeof(struct smscore_idlist_t), GFP_KERNEL);
+       listentry = kzalloc(sizeof(*listentry), GFP_KERNEL);
        if (!listentry)
                return -ENOMEM;
 
@@ -1721,7 +1719,7 @@ int smscore_register_client(struct smscore_device_t *coredev,
                return -EEXIST;
        }
 
-       newclient = kzalloc(sizeof(struct smscore_client_t), GFP_KERNEL);
+       newclient = kzalloc(sizeof(*newclient), GFP_KERNEL);
        if (!newclient)
                return -ENOMEM;
 

diff --git a/drivers/media/dvb-frontends/as102_fe.c b/drivers/media/dvb-frontends/as102_fe.c
index 1fb4ab21d78670dfcb9a8a54738baa838f9b3cf0..b1c84ee914f07bc317823c356e7a28a5a9390586 100644 (file)
--- a/drivers/media/dvb-frontends/as102_fe.c
+++ b/drivers/media/dvb-frontends/as102_fe.c
@@ -455,7 +455,7 @@ struct dvb_frontend *as102_attach(const char *name,
        struct as102_state *state;
        struct dvb_frontend *fe;
 
-       state = kzalloc(sizeof(struct as102_state), GFP_KERNEL);
+       state = kzalloc(sizeof(*state), GFP_KERNEL);
        if (!state)
                return NULL;
 

diff --git a/drivers/media/dvb-frontends/cx24113.c b/drivers/media/dvb-frontends/cx24113.c
index 8fc7333c76b70c532118367ac0204c44df5bf77b..2c5502cab5e1fe2429f4ff883156e1dc8d5ea77a 100644 (file)
--- a/drivers/media/dvb-frontends/cx24113.c
+++ b/drivers/media/dvb-frontends/cx24113.c
@@ -552,8 +552,7 @@ struct dvb_frontend *cx24113_attach(struct dvb_frontend *fe,
                const struct cx24113_config *config, struct i2c_adapter *i2c)
 {
        /* allocate memory for the internal state */
-       struct cx24113_state *state =
-               kzalloc(sizeof(struct cx24113_state), GFP_KERNEL);
+       struct cx24113_state *state = kzalloc(sizeof(*state), GFP_KERNEL);
        int rc;
 
        if (!state)

diff --git a/drivers/media/dvb-frontends/cx24116.c b/drivers/media/dvb-frontends/cx24116.c
index e5135fbe02970c403be28f133675b831a56955c0..531c5861a27e072652cb964e9a0c7fb16a8b68e9 100644 (file)
--- a/drivers/media/dvb-frontends/cx24116.c
+++ b/drivers/media/dvb-frontends/cx24116.c
@@ -1126,7 +1126,7 @@ struct dvb_frontend *cx24116_attach(const struct cx24116_config *config,
        dprintk("%s\n", __func__);
 
        /* allocate memory for the internal state */
-       state = kzalloc(sizeof(struct cx24116_state), GFP_KERNEL);
+       state = kzalloc(sizeof(*state), GFP_KERNEL);
        if (state == NULL)
                goto error1;
 

diff --git a/drivers/media/dvb-frontends/ds3000.c b/drivers/media/dvb-frontends/ds3000.c
index 293c4103e5ff39f46eaf89124afb9b79f4948330..3e347d03acb3abd4a7838e4c313e09aa6c741434 100644 (file)
--- a/drivers/media/dvb-frontends/ds3000.c
+++ b/drivers/media/dvb-frontends/ds3000.c
@@ -839,7 +839,7 @@ struct dvb_frontend *ds3000_attach(const struct ds3000_config *config,
        dprintk("%s\n", __func__);
 
        /* allocate memory for the internal state */
-       state = kzalloc(sizeof(struct ds3000_state), GFP_KERNEL);
+       state = kzalloc(sizeof(*state), GFP_KERNEL);
        if (!state)
                goto error2;
 

diff --git a/drivers/media/dvb-frontends/mb86a20s.c b/drivers/media/dvb-frontends/mb86a20s.c
index 340984100aecb9989db2b7566bb50840b05cf6ef..ba7a433dd42476bc4dba2e2d67cada3bbb13ca62 100644 (file)
--- a/drivers/media/dvb-frontends/mb86a20s.c
+++ b/drivers/media/dvb-frontends/mb86a20s.c
@@ -2071,7 +2071,7 @@ struct dvb_frontend *mb86a20s_attach(const struct mb86a20s_config *config,
        dev_dbg(&i2c->dev, "%s called.\n", __func__);
 
        /* allocate memory for the internal state */
-       state = kzalloc(sizeof(struct mb86a20s_state), GFP_KERNEL);
+       state = kzalloc(sizeof(*state), GFP_KERNEL);
        if (!state)
                goto error;
 

diff --git a/drivers/media/dvb-frontends/sp2.c b/drivers/media/dvb-frontends/sp2.c
index d3b4f882209667e60bee1cd451594c7939ce94d6..dd556012ceb65b340ce464e19e95b7837e053094 100644 (file)
--- a/drivers/media/dvb-frontends/sp2.c
+++ b/drivers/media/dvb-frontends/sp2.c
@@ -381,7 +381,7 @@ static int sp2_probe(struct i2c_client *client,
 
        dev_dbg(&client->dev, "\n");
 
-       s = kzalloc(sizeof(struct sp2), GFP_KERNEL);
+       s = kzalloc(sizeof(*s), GFP_KERNEL);
        if (!s) {
                ret = -ENOMEM;
                goto err;

diff --git a/drivers/media/i2c/adv7842.c b/drivers/media/i2c/adv7842.c
index c582bcb782a614df4073669ae6f966a264eccf79..136aa80a834b022aa31910be6596d978836a6d6a 100644 (file)
--- a/drivers/media/i2c/adv7842.c
+++ b/drivers/media/i2c/adv7842.c
@@ -3467,7 +3467,7 @@ static int adv7842_probe(struct i2c_client *client,
                return -ENODEV;
        }
 
-       state = devm_kzalloc(&client->dev, sizeof(struct adv7842_state), GFP_KERNEL);
+       state = devm_kzalloc(&client->dev, sizeof(*state), GFP_KERNEL);
        if (!state)
                return -ENOMEM;
 

diff --git a/drivers/media/pci/cx18/cx18-driver.c b/drivers/media/pci/cx18/cx18-driver.c
index 9e99c6ef1476be04f56a80469b06762eb917ece6..6efa93168059b7eaa78b90be5e64c6912819789f 100644 (file)
--- a/drivers/media/pci/cx18/cx18-driver.c
+++ b/drivers/media/pci/cx18/cx18-driver.c
@@ -909,7 +909,7 @@ static int cx18_probe(struct pci_dev *pci_dev,
                return -ENOMEM;
        }
 
-       cx = kzalloc(sizeof(struct cx18), GFP_ATOMIC);
+       cx = kzalloc(sizeof(*cx), GFP_ATOMIC);
        if (!cx)
                return -ENOMEM;
 

diff --git a/drivers/media/pci/mantis/hopper_cards.c b/drivers/media/pci/mantis/hopper_cards.c
index a96ac0144fd20ea625f211674657b1889652f182..ecb97dc381fb4ee9359310b68c503393d83124c5 100644 (file)
--- a/drivers/media/pci/mantis/hopper_cards.c
+++ b/drivers/media/pci/mantis/hopper_cards.c
@@ -163,7 +163,7 @@ static int hopper_pci_probe(struct pci_dev *pdev,
        struct mantis_hwconfig *config;
        int err;
 
-       mantis = kzalloc(sizeof(struct mantis_pci), GFP_KERNEL);
+       mantis = kzalloc(sizeof(*mantis), GFP_KERNEL);
        if (mantis == NULL) {
                err = -ENOMEM;
                goto fail0;

diff --git a/drivers/media/pci/mantis/mantis_cards.c b/drivers/media/pci/mantis/mantis_cards.c
index 9958b6f4405f10f3db7d3753e33be3385d092241..4ce8a90d69dcd54ace659944400bd3a81e635826 100644 (file)
--- a/drivers/media/pci/mantis/mantis_cards.c
+++ b/drivers/media/pci/mantis/mantis_cards.c
@@ -173,7 +173,7 @@ static int mantis_pci_probe(struct pci_dev *pdev,
        struct mantis_hwconfig *config;
        int err;
 
-       mantis = kzalloc(sizeof(struct mantis_pci), GFP_KERNEL);
+       mantis = kzalloc(sizeof(*mantis), GFP_KERNEL);
        if (!mantis)
                return -ENOMEM;
 

diff --git a/drivers/media/pci/saa7146/hexium_gemini.c b/drivers/media/pci/saa7146/hexium_gemini.c
index 32c19bad9a17a130cd405dc26068e1d7406c2acc..d31a2d4494d1a8c20778db9c7cd0496309dbabf9 100644 (file)
--- a/drivers/media/pci/saa7146/hexium_gemini.c
+++ b/drivers/media/pci/saa7146/hexium_gemini.c
@@ -260,7 +260,7 @@ static int hexium_attach(struct saa7146_dev *dev, struct saa7146_pci_extension_d
 
        DEB_EE("\n");
 
-       hexium = kzalloc(sizeof(struct hexium), GFP_KERNEL);
+       hexium = kzalloc(sizeof(*hexium), GFP_KERNEL);
        if (!hexium)
                return -ENOMEM;
 

diff --git a/drivers/media/pci/saa7146/hexium_orion.c b/drivers/media/pci/saa7146/hexium_orion.c
index 8d1dc8eea98840b62aebd67e002596ce637542f3..7dc182202f8b27223c15784f2d0f155f15d8070f 100644 (file)
--- a/drivers/media/pci/saa7146/hexium_orion.c
+++ b/drivers/media/pci/saa7146/hexium_orion.c
@@ -219,7 +219,7 @@ static int hexium_probe(struct saa7146_dev *dev)
                return -EFAULT;
        }
 
-       hexium = kzalloc(sizeof(struct hexium), GFP_KERNEL);
+       hexium = kzalloc(sizeof(*hexium), GFP_KERNEL);
        if (!hexium)
                return -ENOMEM;
 

diff --git a/drivers/media/pci/saa7164/saa7164-buffer.c b/drivers/media/pci/saa7164/saa7164-buffer.c
index 6bd665ea190d274dfc096f2d12d36620bf49f25d..c83b2e914dcbf5ada71a92d408a1cbd8992cc565 100644 (file)
--- a/drivers/media/pci/saa7164/saa7164-buffer.c
+++ b/drivers/media/pci/saa7164/saa7164-buffer.c
@@ -98,7 +98,7 @@ struct saa7164_buffer *saa7164_buffer_alloc(struct saa7164_port *port,
                goto ret;
        }
 
-       buf = kzalloc(sizeof(struct saa7164_buffer), GFP_KERNEL);
+       buf = kzalloc(sizeof(*buf), GFP_KERNEL);
        if (!buf)
                goto ret;
 
@@ -281,7 +281,7 @@ struct saa7164_user_buffer *saa7164_buffer_alloc_user(struct saa7164_dev *dev,
 {
        struct saa7164_user_buffer *buf;
 
-       buf = kzalloc(sizeof(struct saa7164_user_buffer), GFP_KERNEL);
+       buf = kzalloc(sizeof(*buf), GFP_KERNEL);
        if (!buf)
                return NULL;
 

diff --git a/drivers/media/platform/atmel/atmel-isc.c b/drivers/media/platform/atmel/atmel-isc.c
index db410c558a74d247623684868fa1dc4daafd4f4e..755d2d4dd142196ae7f2b6eca31a4b930b03ea69 100644 (file)
--- a/drivers/media/platform/atmel/atmel-isc.c
+++ b/drivers/media/platform/atmel/atmel-isc.c
@@ -1505,7 +1505,7 @@ static int isc_formats_init(struct isc_device *isc)
 
        isc->num_user_formats = num_fmts;
        isc->user_formats = devm_kcalloc(isc->dev,
-                                        num_fmts, sizeof(struct isc_format *),
+                                        num_fmts, sizeof(*isc->user_formats),
                                         GFP_KERNEL);
        if (!isc->user_formats)
                return -ENOMEM;

diff --git a/drivers/media/usb/zr364xx/zr364xx.c b/drivers/media/usb/zr364xx/zr364xx.c
index b50cff4ef9eb8c411cc39926f686f53ec1ec9d35..b3d05027bd9a7b858b5bacfe6300fb537d82edc8 100644 (file)
--- a/drivers/media/usb/zr364xx/zr364xx.c
+++ b/drivers/media/usb/zr364xx/zr364xx.c
@@ -1421,7 +1421,7 @@ static int zr364xx_probe(struct usb_interface *intf,
                 le16_to_cpu(udev->descriptor.idVendor),
                 le16_to_cpu(udev->descriptor.idProduct));
 
-       cam = kzalloc(sizeof(struct zr364xx_camera), GFP_KERNEL);
+       cam = kzalloc(sizeof(*cam), GFP_KERNEL);
        if (!cam)
                return -ENOMEM;