RepoMirrors
/
openwrt
mirror of git://git.openwrt.org/openwrt/openwrt.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

tplink-safeloader: fix compilation warnings 

Signed-off-by: Ilya Gordeev <Mirraz@users.noreply.github.com>
This commit is contained in:
Ilya Gordeev 2019-10-12 23:10:09 +05:00 committed by Hauke Mehrtens
parent 53b73131e9
commit 5daf094353
1 changed files with 13 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
tools/firmware-utils/src/tplink-safeloader.c
View File

@ -1684,11 +1684,12 @@ static struct image_partition_entry read_file(const char *part_name, const char
size_t len = statbuf.st_size; size_t len = statbuf.st_size;
if (add_jffs2_eof) if (add_jffs2_eof) {
if (file_system_partition) if (file_system_partition)
len = ALIGN(len + file_system_partition->base, 0x10000) + sizeof(jffs2_eof_mark) - file_system_partition->base; len = ALIGN(len + file_system_partition->base, 0x10000) + sizeof(jffs2_eof_mark) - file_system_partition->base;
else else
len = ALIGN(len, 0x10000) + sizeof(jffs2_eof_mark); len = ALIGN(len, 0x10000) + sizeof(jffs2_eof_mark);
}
struct image_partition_entry entry = alloc_image_partition(part_name, len); struct image_partition_entry entry = alloc_image_partition(part_name, len);
@ -2038,7 +2039,7 @@ static int add_flash_partition(
unsigned long base, unsigned long base,
unsigned long size) unsigned long size)
{ {
int ptr; size_t ptr;
/* check if the list has a free entry */ /* check if the list has a free entry */
for (ptr = 0; ptr < max_entries; ptr++, part_list++) { for (ptr = 0; ptr < max_entries; ptr++, part_list++) {
if (part_list->name == NULL && if (part_list->name == NULL &&
@ -2091,7 +2092,7 @@ static int read_partition_table(
if (fseek(file, offset, SEEK_SET) < 0) if (fseek(file, offset, SEEK_SET) < 0)
error(1, errno, "Can not seek in the firmware"); error(1, errno, "Can not seek in the firmware");
if (fread(buf, 1, 2048, file) < 0) if (fread(buf, 2048, 1, file) != 1)
error(1, errno, "Can not read fwup-ptn from the firmware"); error(1, errno, "Can not read fwup-ptn from the firmware");
buf[2047] = '\0'; buf[2047] = '\0';
@ -2182,18 +2183,18 @@ static void write_partition(
fseek(input_file, entry->base + firmware_offset, SEEK_SET); fseek(input_file, entry->base + firmware_offset, SEEK_SET);
for (offset = 0; sizeof(buf) + offset <= entry->size; offset += sizeof(buf)) { for (offset = 0; sizeof(buf) + offset <= entry->size; offset += sizeof(buf)) {
if (fread(buf, sizeof(buf), 1, input_file) < 0) if (fread(buf, sizeof(buf), 1, input_file) != 1)
error(1, errno, "Can not read partition from input_file"); error(1, errno, "Can not read partition from input_file");
if (fwrite(buf, sizeof(buf), 1, output_file) < 0) if (fwrite(buf, sizeof(buf), 1, output_file) != 1)
error(1, errno, "Can not write partition to output_file"); error(1, errno, "Can not write partition to output_file");
} }
/* write last chunk smaller than buffer */ /* write last chunk smaller than buffer */
if (offset < entry->size) { if (offset < entry->size) {
offset = entry->size - offset; offset = entry->size - offset;
if (fread(buf, offset, 1, input_file) < 0) if (fread(buf, offset, 1, input_file) != 1)
error(1, errno, "Can not read partition from input_file"); error(1, errno, "Can not read partition from input_file");
if (fwrite(buf, offset, 1, output_file) < 0) if (fwrite(buf, offset, 1, output_file) != 1)
error(1, errno, "Can not write partition to output_file"); error(1, errno, "Can not write partition to output_file");
} }
} }
@ -2246,7 +2247,7 @@ static int extract_firmware(const char *input, const char *output_directory)
error(1, 0, "Error can not read the partition table (fwup-ptn)"); error(1, 0, "Error can not read the partition table (fwup-ptn)");
} }
for (int i = 0; i < max_entries; i++) { for (size_t i = 0; i < max_entries; i++) {
if (entries[i].name == NULL && if (entries[i].name == NULL &&
entries[i].base == 0 && entries[i].base == 0 &&
entries[i].size == 0) entries[i].size == 0)
@ -2262,7 +2263,7 @@ static struct flash_partition_entry *find_partition(
struct flash_partition_entry *entries, size_t max_entries, struct flash_partition_entry *entries, size_t max_entries,
const char *name, const char *error_msg) const char *name, const char *error_msg)
{ {
for (int i = 0; i < max_entries; i++, entries++) { for (size_t i = 0; i < max_entries; i++, entries++) {
if (strcmp(entries->name, name) == 0) if (strcmp(entries->name, name) == 0)
return entries; return entries;
} }
@ -2274,19 +2275,19 @@ static struct flash_partition_entry *find_partition(
static void write_ff(FILE *output_file, size_t size) static void write_ff(FILE *output_file, size_t size)
{ {
char buf[4096]; char buf[4096];
int offset; size_t offset;
memset(buf, 0xff, sizeof(buf)); memset(buf, 0xff, sizeof(buf));
for (offset = 0; offset + sizeof(buf) < size ; offset += sizeof(buf)) { for (offset = 0; offset + sizeof(buf) < size ; offset += sizeof(buf)) {
if (fwrite(buf, sizeof(buf), 1, output_file) < 0) if (fwrite(buf, sizeof(buf), 1, output_file) != 1)
error(1, errno, "Can not write 0xff to output_file"); error(1, errno, "Can not write 0xff to output_file");
} }
/* write last chunk smaller than buffer */ /* write last chunk smaller than buffer */
if (offset < size) { if (offset < size) {
offset = size - offset; offset = size - offset;
if (fwrite(buf, offset, 1, output_file) < 0) if (fwrite(buf, offset, 1, output_file) != 1)
error(1, errno, "Can not write partition to output_file"); error(1, errno, "Can not write partition to output_file");
} }
} }