aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBlue Swirl <blauwirbel@gmail.com>2010-04-25 15:27:14 +0000
committerBlue Swirl <blauwirbel@gmail.com>2010-04-25 15:27:14 +0000
commit88bf79500b7659aa7978b445393479d76e9c0c9e (patch)
tree6c2f8fd851140f0c3caea9f67a7715bea112e004
parentf64052478e3828393aea4248da4047b86d48f83f (diff)
downloadqemu-88bf79500b7659aa7978b445393479d76e9c0c9e.zip
qemu-88bf79500b7659aa7978b445393479d76e9c0c9e.tar.gz
qemu-88bf79500b7659aa7978b445393479d76e9c0c9e.tar.bz2
Fix dead initialization, spotted by clang analyzer
Fix clang warnings: /src/qemu/block/vvfat.c:1102:9: warning: Value stored to 'index3' during its initialization is never read int index3=index1+1; /src/qemu/cmd.c:290:15: warning: Value stored to 'p' during its initialization is never read char *p = result; Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
-rw-r--r--block/vvfat.c2
-rw-r--r--cmd.c3
2 files changed, 3 insertions, 2 deletions
diff --git a/block/vvfat.c b/block/vvfat.c
index 0701df4..66259b4 100644
--- a/block/vvfat.c
+++ b/block/vvfat.c
@@ -1099,8 +1099,8 @@ static inline void vvfat_close_current_file(BDRVVVFATState *s)
*/
static inline int find_mapping_for_cluster_aux(BDRVVVFATState* s,int cluster_num,int index1,int index2)
{
- int index3=index1+1;
while(1) {
+ int index3;
mapping_t* mapping;
index3=(index1+index2)/2;
mapping=array_get(&(s->mapping),index3);
diff --git a/cmd.c b/cmd.c
index cc70311..2336334 100644
--- a/cmd.c
+++ b/cmd.c
@@ -287,7 +287,8 @@ static char *qemu_strsep(char **input, const char *delim)
{
char *result = *input;
if (result != NULL) {
- char *p = result;
+ char *p;
+
for (p = result; *p != '\0'; p++) {
if (strchr(delim, *p)) {
break;