firmware-utils: fix -Wmaybe-uninitialized warnings
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
This commit is contained in:
		| @@ -150,7 +150,8 @@ static void usage() { | |||||||
| int main(int argc, char **argv) { | int main(int argc, char **argv) { | ||||||
| 	struct trx_header hdr; | 	struct trx_header hdr; | ||||||
| 	struct asustrx_tail tail = { }; | 	struct asustrx_tail tail = { }; | ||||||
| 	FILE *in, *out; | 	FILE *out = NULL; | ||||||
|  | 	FILE *in = NULL; | ||||||
| 	uint8_t buf[1024]; | 	uint8_t buf[1024]; | ||||||
| 	size_t bytes; | 	size_t bytes; | ||||||
| 	size_t length = 0; | 	size_t length = 0; | ||||||
|   | |||||||
| @@ -554,6 +554,10 @@ csum_get(struct csum_state *css) | |||||||
| 		break; | 		break; | ||||||
| 	case CSUM_TYPE_32: | 	case CSUM_TYPE_32: | ||||||
| 		ret = csum32_get(css); | 		ret = csum32_get(css); | ||||||
|  | 		break; | ||||||
|  | 	default: | ||||||
|  | 		ERR("invalid checksum size\n"); | ||||||
|  | 		return 0; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	return ret; | 	return ret; | ||||||
| @@ -755,6 +759,9 @@ write_out_header(FILE *outfile) | |||||||
| 		res = write_out_data(outfile, (uint8_t *)&tmp.nfs, | 		res = write_out_data(outfile, (uint8_t *)&tmp.nfs, | ||||||
| 					sizeof(tmp.nfs), NULL); | 					sizeof(tmp.nfs), NULL); | ||||||
| 		break; | 		break; | ||||||
|  | 	default: | ||||||
|  | 		ERR("invalid header type\n"); | ||||||
|  | 		return -EINVAL; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	return res; | 	return res; | ||||||
|   | |||||||
| @@ -493,6 +493,9 @@ csum_get(struct csum_state *css) | |||||||
| 	case CSUM_SIZE_16: | 	case CSUM_SIZE_16: | ||||||
| 		ret = csum16_get(css); | 		ret = csum16_get(css); | ||||||
| 		break; | 		break; | ||||||
|  | 	default: | ||||||
|  | 		ERR("invalid checksum size\n"); | ||||||
|  | 		return 0; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	return ret; | 	return ret; | ||||||
|   | |||||||
| @@ -68,7 +68,8 @@ main(int ac, char *av[]) | |||||||
| 	int fixmode = 0; | 	int fixmode = 0; | ||||||
| 	int have_regionversion = 0; | 	int have_regionversion = 0; | ||||||
|  |  | ||||||
| 	FILE *ifile, *ofile; | 	FILE *ifile = NULL; | ||||||
|  | 	FILE *ofile = NULL; | ||||||
| 	int c; | 	int c; | ||||||
| 	uint32_t cksum; | 	uint32_t cksum; | ||||||
| 	uint32_t bcnt; | 	uint32_t bcnt; | ||||||
|   | |||||||
| @@ -92,7 +92,8 @@ main(int ac, char *av[]) | |||||||
| 	char region[MAX_REG_LEN]; | 	char region[MAX_REG_LEN]; | ||||||
| 	int kernel = 0; | 	int kernel = 0; | ||||||
|  |  | ||||||
| 	FILE *ifile, *ofile; | 	FILE *ifile = NULL; | ||||||
|  | 	FILE *ofile = NULL; | ||||||
| 	int c; | 	int c; | ||||||
|  |  | ||||||
| 	uint32_t cksum; | 	uint32_t cksum; | ||||||
|   | |||||||
| @@ -251,7 +251,8 @@ int | |||||||
| main(int argc, char **argv) | main(int argc, char **argv) | ||||||
| { | { | ||||||
| 	int 		opt; | 	int 		opt; | ||||||
| 	char 		*filename, *progname; | 	char 		*filename = NULL; | ||||||
|  | 	char		*progname; | ||||||
| 	op_mode_t	opmode = NONE; | 	op_mode_t	opmode = NONE; | ||||||
|  |  | ||||||
| 	progname = argv[0]; | 	progname = argv[0]; | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Rafał Miłecki
					Rafał Miłecki