lkml.org 
[lkml]   [2005]   [Mar]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH][1/2] isofs: kfree handles NULL pointers fine (rock.c)

kfree() has no trouble being handed a NULL pointer, so checking for one
before calling it is redundant. This patch removes the redundant checks in
fs/isofs/rock.c


Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>

diff -up linux-2.6.11-mm4-orig/fs/isofs/rock.c linux-2.6.11-mm4/fs/isofs/rock.c
--- linux-2.6.11-mm4-orig/fs/isofs/rock.c 2005-03-02 08:38:10.000000000 +0100
+++ linux-2.6.11-mm4/fs/isofs/rock.c 2005-03-16 23:36:30.000000000 +0100
@@ -62,7 +62,7 @@
}

#define MAYBE_CONTINUE(LABEL,DEV) \
- {if (buffer) { kfree(buffer); buffer = NULL; } \
+ {kfree(buffer); buffer = NULL; \
if (cont_extent){ \
int block, offset, offset1; \
struct buffer_head * pbh; \
@@ -145,7 +145,7 @@ int get_rock_ridge_filename(struct iso_d
retnamlen += rr->len - 5;
break;
case SIG('R','E'):
- if (buffer) kfree(buffer);
+ kfree(buffer);
return -1;
default:
break;
@@ -153,10 +153,10 @@ int get_rock_ridge_filename(struct iso_d
}
}
MAYBE_CONTINUE(repeat,inode);
- if (buffer) kfree(buffer);
+ kfree(buffer);
return retnamlen; /* If 0, this file did not have a NM field */
out:
- if(buffer) kfree(buffer);
+ kfree(buffer);
return 0;
}

@@ -355,7 +355,7 @@ parse_rock_ridge_inode_internal(struct i
}
MAYBE_CONTINUE(repeat,inode);
out:
- if(buffer) kfree(buffer);
+ kfree(buffer);
return 0;
}

@@ -517,8 +517,7 @@ static int rock_ridge_symlink_readpage(s
}
}
MAYBE_CONTINUE(repeat, inode);
- if (buffer)
- kfree(buffer);
+ kfree(buffer);

if (rpnt == link)
goto fail;
@@ -532,8 +531,7 @@ static int rock_ridge_symlink_readpage(s

/* error exit from macro */
out:
- if (buffer)
- kfree(buffer);
+ kfree(buffer);
goto fail;
out_noread:
printk("unable to read i-node block");

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:11    [W:0.021 / U:0.748 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site