From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Valentin Vidic <vvidic@valentin-vidic.from.hr>
Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu>,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: exfat: cleanup explicit comparisons to NULL
Date: Tue, 3 Sep 2019 22:09:37 +0200 [thread overview]
Message-ID: <20190903200937.GA18191@kroah.com> (raw)
In-Reply-To: <20190903171337.22889-1-vvidic@valentin-vidic.from.hr>
On Tue, Sep 03, 2019 at 07:13:37PM +0200, Valentin Vidic wrote:
> Fixes checkpatch.pl warnings:
>
> CHECK: Comparison to NULL could be written "expr"
> CHECK: Comparison to NULL could be written "!expr"
>
> Signed-off-by: Valentin Vidic <vvidic@valentin-vidic.from.hr>
> ---
> drivers/staging/exfat/exfat_core.c | 34 ++++++++---------
> drivers/staging/exfat/exfat_super.c | 58 ++++++++++++++---------------
> 2 files changed, 46 insertions(+), 46 deletions(-)
>
> diff --git a/drivers/staging/exfat/exfat_core.c b/drivers/staging/exfat/exfat_core.c
> index 46b9f4455da1..7b39544cdaf1 100644
> --- a/drivers/staging/exfat/exfat_core.c
> +++ b/drivers/staging/exfat/exfat_core.c
> @@ -100,7 +100,7 @@ void fs_set_vol_flags(struct super_block *sb, u32 new_flag)
> p_fs->vol_flag = new_flag;
>
> if (p_fs->vol_type == EXFAT) {
> - if (p_fs->pbr_bh == NULL) {
> + if (!p_fs->pbr_bh) {
> if (sector_read(sb, p_fs->PBR_sector,
> &p_fs->pbr_bh, 1) != FFS_SUCCESS)
> return;
> @@ -543,7 +543,7 @@ s32 load_alloc_bitmap(struct super_block *sb)
> p_fs->vol_amap = kmalloc_array(p_fs->map_sectors,
> sizeof(struct buffer_head *),
> GFP_KERNEL);
> - if (p_fs->vol_amap == NULL)
> + if (!p_fs->vol_amap)
> return FFS_MEMORYERR;
>
> sector = START_SECTOR(p_fs->map_clu);
> @@ -685,7 +685,7 @@ void sync_alloc_bitmap(struct super_block *sb)
> int i;
> struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
>
> - if (p_fs->vol_amap == NULL)
> + if (!p_fs->vol_amap)
> return;
>
> for (i = 0; i < p_fs->map_sectors; i++)
> @@ -714,7 +714,7 @@ static s32 __load_upcase_table(struct super_block *sb, sector_t sector,
>
> upcase_table = p_fs->vol_utbl = kmalloc(UTBL_COL_COUNT * sizeof(u16 *),
> GFP_KERNEL);
> - if (upcase_table == NULL)
> + if (!upcase_table)
> return FFS_MEMORYERR;
> memset(upcase_table, 0, UTBL_COL_COUNT * sizeof(u16 *));
>
> @@ -750,11 +750,11 @@ static s32 __load_upcase_table(struct super_block *sb, sector_t sector,
> else { /* uni != index , uni != 0xFFFF */
> u16 col_index = get_col_index(index);
>
> - if (upcase_table[col_index] == NULL) {
> + if (!upcase_table[col_index]) {
> pr_debug("alloc = 0x%X\n", col_index);
> upcase_table[col_index] = kmalloc_array(UTBL_ROW_COUNT,
> sizeof(u16), GFP_KERNEL);
> - if (upcase_table[col_index] == NULL) {
> + if (!upcase_table[col_index]) {
> ret = FFS_MEMORYERR;
> goto error;
> }
> @@ -794,7 +794,7 @@ static s32 __load_default_upcase_table(struct super_block *sb)
>
> upcase_table = p_fs->vol_utbl = kmalloc(UTBL_COL_COUNT * sizeof(u16 *),
> GFP_KERNEL);
> - if (upcase_table == NULL)
> + if (!upcase_table)
> return FFS_MEMORYERR;
> memset(upcase_table, 0, UTBL_COL_COUNT * sizeof(u16 *));
>
> @@ -812,12 +812,12 @@ static s32 __load_default_upcase_table(struct super_block *sb)
> else { /* uni != index , uni != 0xFFFF */
> u16 col_index = get_col_index(index);
>
> - if (upcase_table[col_index] == NULL) {
> + if (!upcase_table[col_index]) {
> pr_debug("alloc = 0x%X\n", col_index);
> upcase_table[col_index] = kmalloc_array(UTBL_ROW_COUNT,
> sizeof(u16),
> GFP_KERNEL);
> - if (upcase_table[col_index] == NULL) {
> + if (!upcase_table[col_index]) {
> ret = FFS_MEMORYERR;
> goto error;
> }
> @@ -1640,7 +1640,7 @@ struct dentry_t *get_entry_with_sector(struct super_block *sb, sector_t sector,
>
> buf = buf_getblk(sb, sector);
>
> - if (buf == NULL)
> + if (!buf)
> return NULL;
>
> return (struct dentry_t *)(buf + offset);
> @@ -1658,10 +1658,10 @@ struct dentry_t *get_entry_in_dir(struct super_block *sb, struct chain_t *p_dir,
>
> buf = buf_getblk(sb, sec);
>
> - if (buf == NULL)
> + if (!buf)
> return NULL;
>
> - if (sector != NULL)
> + if (sector)
> *sector = sec;
> return (struct dentry_t *)(buf + off);
> }
> @@ -1721,7 +1721,7 @@ struct entry_set_cache_t *get_entry_set_in_dir(struct super_block *sb,
> sec += START_SECTOR(clu);
>
> buf = buf_getblk(sb, sec);
> - if (buf == NULL)
> + if (!buf)
> goto err_out;
>
> ep = (struct dentry_t *)(buf + off);
> @@ -1741,7 +1741,7 @@ struct entry_set_cache_t *get_entry_set_in_dir(struct super_block *sb,
> pr_debug("%s: trying to kmalloc %zx bytes for %d entries\n", __func__,
> bufsize, num_entries);
> es = kmalloc(bufsize, GFP_KERNEL);
> - if (es == NULL)
> + if (!es)
> goto err_out;
>
> es->num_entries = num_entries;
> @@ -1820,7 +1820,7 @@ struct entry_set_cache_t *get_entry_set_in_dir(struct super_block *sb,
> sec++;
> }
> buf = buf_getblk(sb, sec);
> - if (buf == NULL)
> + if (!buf)
> goto err_out;
> off = 0;
> ep = (struct dentry_t *)(buf);
> @@ -1872,7 +1872,7 @@ static s32 __write_partial_entries_in_entry_set(struct super_block *sb,
> remaining_byte_in_sector >> DENTRY_SIZE_BITS,
> num_entries);
> buf = buf_getblk(sb, sec);
> - if (buf == NULL)
> + if (!buf)
> goto err_out;
> pr_debug("es->buf %p buf_off %u\n", esbuf, buf_off);
> pr_debug("copying %d entries from %p to sector %llu\n",
> @@ -2649,7 +2649,7 @@ void exfat_get_uni_name_from_ext_entry(struct super_block *sb,
> struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
>
> es = get_entry_set_in_dir(sb, p_dir, entry, ES_ALL_ENTRIES, &ep);
> - if (es == NULL || es->num_entries < 3) {
> + if (!es || es->num_entries < 3) {
> if (es)
> release_entry_set(es);
> return;
> diff --git a/drivers/staging/exfat/exfat_super.c b/drivers/staging/exfat/exfat_super.c
> index 881cd85cf677..8d93403a3308 100644
> --- a/drivers/staging/exfat/exfat_super.c
> +++ b/drivers/staging/exfat/exfat_super.c
> @@ -341,7 +341,7 @@ static int exfat_cmpi(const struct dentry *dentry, unsigned int len,
> alen = exfat_striptail_len(name);
> blen = __exfat_striptail_len(len, str);
> if (alen == blen) {
> - if (t == NULL) {
> + if (!t) {
> if (strncasecmp(name->name, str, alen) == 0)
> return 0;
> } else if (nls_strnicmp(t, name->name, str, alen) == 0)
> @@ -587,7 +587,7 @@ static int ffsGetVolInfo(struct super_block *sb, struct vol_info_t *info)
> struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
>
> /* check the validity of pointer parameters */
> - if (info == NULL)
> + if (!info)
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -650,7 +650,7 @@ static int ffsLookupFile(struct inode *inode, char *path, struct file_id_t *fid)
> pr_debug("%s entered\n", __func__);
>
> /* check the validity of pointer parameters */
> - if ((fid == NULL) || (path == NULL) || (*path == '\0'))
> + if (!fid || !path || (*path == '\0'))
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -743,7 +743,7 @@ static int ffsCreateFile(struct inode *inode, char *path, u8 mode,
> int ret;
>
> /* check the validity of pointer parameters */
> - if ((fid == NULL) || (path == NULL) || (*path == '\0'))
> + if (!fid || !path || (*path == '\0'))
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -788,11 +788,11 @@ static int ffsReadFile(struct inode *inode, struct file_id_t *fid, void *buffer,
> struct bd_info_t *p_bd = &(EXFAT_SB(sb)->bd_info);
>
> /* check the validity of the given file id */
> - if (fid == NULL)
> + if (!fid)
> return FFS_INVALIDFID;
>
> /* check the validity of pointer parameters */
> - if (buffer == NULL)
> + if (!buffer)
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -811,7 +811,7 @@ static int ffsReadFile(struct inode *inode, struct file_id_t *fid, void *buffer,
> count = fid->size - fid->rwoffset;
>
> if (count == 0) {
> - if (rcount != NULL)
> + if (rcount)
> *rcount = 0;
> ret = FFS_EOF;
> goto out;
> @@ -885,7 +885,7 @@ static int ffsReadFile(struct inode *inode, struct file_id_t *fid, void *buffer,
> /* How did this ever work and not leak a brlse()?? */
> err_out:
> /* set the size of read bytes */
> - if (rcount != NULL)
> + if (rcount)
> *rcount = read_bytes;
>
> if (p_fs->dev_ejected)
> @@ -917,11 +917,11 @@ static int ffsWriteFile(struct inode *inode, struct file_id_t *fid,
> struct bd_info_t *p_bd = &(EXFAT_SB(sb)->bd_info);
>
> /* check the validity of the given file id */
> - if (fid == NULL)
> + if (!fid)
> return FFS_INVALIDFID;
>
> /* check the validity of pointer parameters */
> - if (buffer == NULL)
> + if (!buffer)
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -937,7 +937,7 @@ static int ffsWriteFile(struct inode *inode, struct file_id_t *fid,
> fid->rwoffset = fid->size;
>
> if (count == 0) {
> - if (wcount != NULL)
> + if (wcount)
> *wcount = 0;
> ret = FFS_SUCCESS;
> goto out;
> @@ -1096,7 +1096,7 @@ static int ffsWriteFile(struct inode *inode, struct file_id_t *fid,
> if (p_fs->vol_type == EXFAT) {
> es = get_entry_set_in_dir(sb, &(fid->dir), fid->entry,
> ES_ALL_ENTRIES, &ep);
> - if (es == NULL)
> + if (!es)
> goto err_out;
> ep2 = ep+1;
> } else {
> @@ -1138,7 +1138,7 @@ static int ffsWriteFile(struct inode *inode, struct file_id_t *fid,
>
> err_out:
> /* set the size of written bytes */
> - if (wcount != NULL)
> + if (wcount)
> *wcount = write_bytes;
>
> if (num_alloced == 0)
> @@ -1225,7 +1225,7 @@ static int ffsTruncateFile(struct inode *inode, u64 old_size, u64 new_size)
> if (p_fs->vol_type == EXFAT) {
> es = get_entry_set_in_dir(sb, &fid->dir, fid->entry,
> ES_ALL_ENTRIES, &ep);
> - if (es == NULL) {
> + if (!es) {
> ret = FFS_MEDIAERR;
> goto out;
> }
> @@ -1320,11 +1320,11 @@ static int ffsMoveFile(struct inode *old_parent_inode, struct file_id_t *fid,
> s32 new_entry = 0;
>
> /* check the validity of the given file id */
> - if (fid == NULL)
> + if (!fid)
> return FFS_INVALIDFID;
>
> /* check the validity of pointer parameters */
> - if ((new_path == NULL) || (*new_path == '\0'))
> + if (!new_path || (*new_path == '\0'))
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -1441,7 +1441,7 @@ static int ffsRemoveFile(struct inode *inode, struct file_id_t *fid)
> struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
>
> /* check the validity of the given file id */
> - if (fid == NULL)
> + if (!fid)
> return FFS_INVALIDFID;
>
> /* acquire the lock for file system critical section */
> @@ -1529,7 +1529,7 @@ static int ffsSetAttr(struct inode *inode, u32 attr)
> if (p_fs->vol_type == EXFAT) {
> es = get_entry_set_in_dir(sb, &(fid->dir), fid->entry,
> ES_ALL_ENTRIES, &ep);
> - if (es == NULL) {
> + if (!es) {
> ret = FFS_MEDIAERR;
> goto out;
> }
> @@ -1645,7 +1645,7 @@ static int ffsReadStat(struct inode *inode, struct dir_entry_t *info)
> if (p_fs->vol_type == EXFAT) {
> es = get_entry_set_in_dir(sb, &(fid->dir), fid->entry,
> ES_2_ENTRIES, &ep);
> - if (es == NULL) {
> + if (!es) {
> ret = FFS_MEDIAERR;
> goto out;
> }
> @@ -1769,7 +1769,7 @@ static int ffsWriteStat(struct inode *inode, struct dir_entry_t *info)
> if (p_fs->vol_type == EXFAT) {
> es = get_entry_set_in_dir(sb, &(fid->dir), fid->entry,
> ES_ALL_ENTRIES, &ep);
> - if (es == NULL) {
> + if (!es) {
> ret = FFS_MEDIAERR;
> goto out;
> }
> @@ -1838,7 +1838,7 @@ static int ffsMapCluster(struct inode *inode, s32 clu_offset, u32 *clu)
> struct file_id_t *fid = &(EXFAT_I(inode)->fid);
>
> /* check the validity of pointer parameters */
> - if (clu == NULL)
> + if (!clu)
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -1922,7 +1922,7 @@ static int ffsMapCluster(struct inode *inode, s32 clu_offset, u32 *clu)
> if (p_fs->vol_type == EXFAT) {
> es = get_entry_set_in_dir(sb, &fid->dir, fid->entry,
> ES_ALL_ENTRIES, &ep);
> - if (es == NULL) {
> + if (!es) {
> ret = FFS_MEDIAERR;
> goto out;
> }
> @@ -1990,7 +1990,7 @@ static int ffsCreateDir(struct inode *inode, char *path, struct file_id_t *fid)
> pr_debug("%s entered\n", __func__);
>
> /* check the validity of pointer parameters */
> - if ((fid == NULL) || (path == NULL) || (*path == '\0'))
> + if (!fid || !path || (*path == '\0'))
> return FFS_ERROR;
>
> /* acquire the lock for file system critical section */
> @@ -2036,7 +2036,7 @@ static int ffsReadDir(struct inode *inode, struct dir_entry_t *dir_entry)
> struct file_id_t *fid = &(EXFAT_I(inode)->fid);
>
> /* check the validity of pointer parameters */
> - if (dir_entry == NULL)
> + if (!dir_entry)
> return FFS_ERROR;
>
> /* check if the given file ID is opened */
> @@ -2227,7 +2227,7 @@ static int ffsRemoveDir(struct inode *inode, struct file_id_t *fid)
> struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
>
> /* check the validity of the given file id */
> - if (fid == NULL)
> + if (!fid)
> return FFS_INVALIDFID;
>
> dir.dir = fid->dir.dir;
> @@ -3115,10 +3115,10 @@ static const char *exfat_get_link(struct dentry *dentry, struct inode *inode,
> {
> struct exfat_inode_info *ei = EXFAT_I(inode);
>
> - if (ei->target != NULL) {
> + if (ei->target) {
> char *cookie = ei->target;
>
> - if (cookie != NULL)
> + if (cookie)
> return (char *)(ei->target);
> }
> return NULL;
> @@ -3780,7 +3780,7 @@ static int parse_options(char *options, int silent, int *debug,
> if (!options)
> goto out;
>
> - while ((p = strsep(&options, ",")) != NULL) {
> + while (p = strsep(&options, ",")) {
There was an "extra" set of () in here to keep gcc happy, otherwise we
now have:
drivers/staging/exfat/exfat_super.c: In function parse_options:
drivers/staging/exfat/exfat_super.c:3785:9: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
3785 | while (p = strsep(&options, ",")) {
| ^
So I can't take this patch, sorry.
Please fix up and resend.
thanks,
greg k-h
next prev parent reply other threads:[~2019-09-03 20:09 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-03 17:13 [PATCH] staging: exfat: cleanup explicit comparisons to NULL Valentin Vidic
2019-09-03 20:09 ` Greg Kroah-Hartman [this message]
2019-09-03 20:56 ` [PATCH v2] " Valentin Vidic
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190903200937.GA18191@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=devel@driverdev.osuosl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=valdis.kletnieks@vt.edu \
--cc=vvidic@valentin-vidic.from.hr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.