From 7aa258212909180751d878771965d8d9d7048bf7 Mon Sep 17 00:00:00 2001 From: Thomas Schmitt Date: Thu, 5 Sep 2013 10:01:08 +0200 Subject: [PATCH] Reacted on warnings of PLD Linux build log --- libisofs/ecma119.c | 3 +-- libisofs/hfsplus.c | 11 ++++------- libisofs/image.c | 11 +++++++++-- libisofs/make_isohybrid_mbr.c | 5 +++-- 4 files changed, 17 insertions(+), 13 deletions(-) diff --git a/libisofs/ecma119.c b/libisofs/ecma119.c index 6dde396..15995bc 100644 --- a/libisofs/ecma119.c +++ b/libisofs/ecma119.c @@ -1734,7 +1734,7 @@ int ecma119_image_new(IsoImage *src, IsoWriteOpts *opts, Ecma119Image **img) int sa_type; Ecma119Image *target; IsoImageWriter *writer; - int el_torito_writer_index = -1, file_src_writer_index = -1; + int file_src_writer_index = -1; int system_area_options = 0; char *system_area = NULL; int write_count = 0, write_count_mem; @@ -2073,7 +2073,6 @@ int ecma119_image_new(IsoImage *src, IsoWriteOpts *opts, Ecma119Image **img) if (ret < 0) { goto target_cleanup; } - el_torito_writer_index = target->nwriters - 1; } /* create writer for Joliet structure */ diff --git a/libisofs/hfsplus.c b/libisofs/hfsplus.c index 442923b..45a4b71 100644 --- a/libisofs/hfsplus.c +++ b/libisofs/hfsplus.c @@ -477,8 +477,8 @@ static int hfsplus_writer_compute_data_blocks(IsoImageWriter *writer) { Ecma119Image *t; - uint32_t i, link_blocks, hfsp_curblock; - uint32_t block_fac, cat_node_size, block_size; + uint32_t i, hfsp_curblock; + uint32_t block_fac, block_size; if (writer == NULL) { return ISO_OUT_OF_MEM; @@ -487,7 +487,6 @@ int hfsplus_writer_compute_data_blocks(IsoImageWriter *writer) t = writer->target; block_size = t->hfsp_block_size; block_fac = t->hfsp_iso_block_fac; - cat_node_size = t->hfsp_cat_node_size; iso_msg_debug(t->image->id, "(b) curblock=%d, nodes =%d", t->curblock, t->hfsp_nnodes); t->hfsp_part_start = t->curblock * block_fac; @@ -500,7 +499,7 @@ int hfsplus_writer_compute_data_blocks(IsoImageWriter *writer) t->hfsp_catalog_file_start = hfsp_curblock; /* - hfsp_curblock += (t->hfsp_nnodes * cat_node_size + block_size - 1) / block_size; + hfsp_curblock += (t->hfsp_nnodes * t->hfsp_cat_node_size + block_size - 1) / block_size; */ hfsp_curblock += 2 * t->hfsp_nnodes; @@ -509,7 +508,6 @@ int hfsplus_writer_compute_data_blocks(IsoImageWriter *writer) iso_msg_debug(t->image->id, "(d) hfsp_curblock=%d, nodes =%d", hfsp_curblock, t->hfsp_nnodes); - link_blocks = 0; for (i = 0; i < t->hfsp_nleafs; i++) if (t->hfsp_leafs[i].unix_type == UNIX_SYMLINK) { @@ -1227,7 +1225,7 @@ int update_symlink(Ecma119Image *target, uint32_t changed_idx, char *new_name, char *orig_dest, *orig_start, *orig_end; char *hfsp_dest, *hfsp_start, *hfsp_end; int ret = 0; - unsigned int comp_len, orig_len, hfsp_len, hfsp_comp_len; + unsigned int comp_len, orig_len, hfsp_len; if (target->hfsp_leafs[link_idx].node->type != LIBISO_SYMLINK) return ISO_SUCCESS; @@ -1271,7 +1269,6 @@ int update_symlink(Ecma119Image *target, uint32_t changed_idx, char *new_name, hfsp_end = strchr(hfsp_start, '/'); if (hfsp_end == NULL) hfsp_end = hfsp_start + strlen(hfsp_start); - hfsp_comp_len = hfsp_end - hfsp_start; if (comp_len == 0 || (comp_len == 1 && orig_start[0] == '.')) continue; diff --git a/libisofs/image.c b/libisofs/image.c index a765d25..d830136 100644 --- a/libisofs/image.c +++ b/libisofs/image.c @@ -406,6 +406,7 @@ static int dir_update_size(IsoImage *image, IsoDir *dir) { IsoNode *pos; + int ret; #ifdef Libisofs_update_sizes_abortablE char *path= NULL; @@ -416,7 +417,6 @@ int dir_update_size(IsoImage *image, IsoDir *dir) pos = dir->children; while (pos) { - int ret = 1; if (pos->type == LIBISO_FILE) { ret = iso_stream_update_size(ISO_FILE(pos)->stream); } else if (pos->type == LIBISO_DIR) { @@ -428,6 +428,8 @@ int dir_update_size(IsoImage *image, IsoDir *dir) return ret; /* Message already issued by dir_update_size */ #endif + } else { + ret = 1; } #ifdef Libisofs_update_sizes_abortablE @@ -465,7 +467,12 @@ int dir_update_size(IsoImage *image, IsoDir *dir) return cancel_ret; /* cancel due error threshold */ } -#endif /* Libisofs_update_sizes_abortablE */ +#else + + if (ret < 0) + ret = 1; /* ignore error */ + +#endif /* ! Libisofs_update_sizes_abortablE */ pos = pos->next; } diff --git a/libisofs/make_isohybrid_mbr.c b/libisofs/make_isohybrid_mbr.c index 18e7e86..9d88fd7 100644 --- a/libisofs/make_isohybrid_mbr.c +++ b/libisofs/make_isohybrid_mbr.c @@ -563,7 +563,7 @@ int make_isolinux_mbr(int32_t *img_blocks, Ecma119Image *t, uint32_t id, part, nominal_part_size; off_t hd_img_blocks, hd_boot_lba; char *wpt; - uint32_t boot_lba, mbr_id; + uint32_t boot_lba; int head_count, sector_count, ret; int gpt_count = 0, gpt_idx[128], apm_count = 0, gpt_cursor; /* For generating a weak random number */ @@ -573,7 +573,6 @@ int make_isolinux_mbr(int32_t *img_blocks, Ecma119Image *t, hd_img_blocks = ((off_t) *img_blocks) * (off_t) 4; boot_lba = t->bootsrc[0]->sections[0].block; - mbr_id = 0; head_count = t->partition_heads_per_cyl; sector_count = t->partition_secs_per_head; @@ -608,6 +607,8 @@ int make_isolinux_mbr(int32_t *img_blocks, Ecma119Image *t, gettimeofday(&tv, &tz); id = 0xffffffff & (tv.tv_sec ^ (tv.tv_usec * 2000)); lsb_to_buf(&wpt, id, 32, 0); + } else { + wpt+= 4; } /* write word 0 # Offset 444