[tools] firmware-utils: unify cc and cc2 macros, simplifies Makefile

[openwrt.git] / tools / wrt350nv2-builder / src / wrt350nv2-builder.c

diff --git a/tools/wrt350nv2-builder/src/wrt350nv2-builder.c b/tools/wrt350nv2-builder/src/wrt350nv2-builder.c
index 9bc0006..3693286 100644 (file)
--- a/tools/wrt350nv2-builder/src/wrt350nv2-builder.c
+++ b/tools/wrt350nv2-builder/src/wrt350nv2-builder.c
@@ -73,6 +73,7 @@
 #include <stdlib.h>    // system(), etc.
 #include <string.h>    // basename(), strerror(), strdup(), etc.
 #include <unistd.h>    // optopt(), access(), etc.
 #include <stdlib.h>    // system(), etc.
 #include <string.h>    // basename(), strerror(), strdup(), etc.
 #include <unistd.h>    // optopt(), access(), etc.

+#include <libgen.h>

 #include <sys/wait.h>  // WEXITSTATUS, etc.
 
 // custom includes
 #include <sys/wait.h>  // WEXITSTATUS, etc.
 
 // custom includes


@@ -186,8 +187,8 @@ int parse_par_file(FILE *f_par) {
        int lineno;
        int count;
 
        int lineno;
        int count;
 

-       char *string1;
-       char *string2;
+       char string1[256];
+       char string2[256];

        int value;
 
        mtd_info *mtd;
        int value;
 
        mtd_info *mtd;


@@ -207,7 +208,7 @@ int parse_par_file(FILE *f_par) {
                        }
                        if (buffer == NULL) {
                                exitcode = 1;
                        }
                        if (buffer == NULL) {
                                exitcode = 1;

-                               printf("parse_par_file: can not allocate %i bytes\n", buffer_size);
+                               printf("parse_par_file: can not allocate %i bytes\n", (int) buffer_size);

                                break;
                        }
 
                                break;
                        }
 


@@ -250,15 +251,13 @@ int parse_par_file(FILE *f_par) {
 
                lprintf(DEBUG_LVL2, " line %i (%i) %s", lineno, count, line);
 
 
                lprintf(DEBUG_LVL2, " line %i (%i) %s", lineno, count, line);
 

-               string1 = NULL;
-               string2 = NULL;

                value = 0;
                mtd = NULL;
 
                // split line if starting with a colon
                switch (line[0]) {
                        case ':':
                value = 0;
                mtd = NULL;
 
                // split line if starting with a colon
                switch (line[0]) {
                        case ':':

-                               count = sscanf(line, ":%ms %i %ms", &string1, &value, &string2);
+                               count = sscanf(line, ":%255s %i %255s", string1, &value, string2);

                                if (count != 3) {
                                        printf("line %i does not meet defined format (:<mtdname> <mtdsize> <file>)\n", lineno);
                                } else {
                                if (count != 3) {
                                        printf("line %i does not meet defined format (:<mtdname> <mtdsize> <file>)\n", lineno);
                                } else {


@@ -278,8 +277,7 @@ int parse_par_file(FILE *f_par) {
                                                printf("mtd %s in line %i multiple definitions\n", string1, lineno);
                                        } else {
                                                mtd->size = value;
                                                printf("mtd %s in line %i multiple definitions\n", string1, lineno);
                                        } else {
                                                mtd->size = value;

-                                               mtd->filename = string2;
-                                               string2 = NULL; // do not free
+                                               mtd->filename = strdup(string2);

 
                                                // Get file size
                                                f_in = fopen(mtd->filename, "rb");
 
                                                // Get file size
                                                f_in = fopen(mtd->filename, "rb");


@@ -318,7 +316,7 @@ int parse_par_file(FILE *f_par) {
                                }
                                break;
                        case '#':       // integer values
                                }
                                break;
                        case '#':       // integer values

-                               count = sscanf(line, "#%ms %i", &string1, &value);
+                               count = sscanf(line, "#%255s %i", string1, &value);

                                if (count != 2) {
                                        printf("line %i does not meet defined format (:<variable name> <integer>\n", lineno);
                                } else {
                                if (count != 2) {
                                        printf("line %i does not meet defined format (:<variable name> <integer>\n", lineno);
                                } else {


@@ -334,7 +332,7 @@ int parse_par_file(FILE *f_par) {
                                }
                                break;
                        case '$':       // strings
                                }
                                break;
                        case '$':       // strings

-                               count = sscanf(line, "$%ms %ms", &string1, &string2);
+                               count = sscanf(line, "$%255s %255s", string1, string2);

                                if (count != 2) {
                                        printf("line %i does not meet defined format (:<mtdname> <mtdsize> <file>)\n", lineno);
                                } else {
                                if (count != 2) {
                                        printf("line %i does not meet defined format (:<mtdname> <mtdsize> <file>)\n", lineno);
                                } else {


@@ -352,13 +350,6 @@ int parse_par_file(FILE *f_par) {
                        default:
                                break;
                }
                        default:
                                break;
                }

-
-               if (string1) {
-                       free(string1);
-               }
-               if (string2) {
-                       free(string2);
-               }

        }
        free(buffer);
 
        }
        free(buffer);
 


@@ -561,7 +552,7 @@ int create_zip_file(char *zip_filename, char *bin_filename) {
                }
                if (buffer == NULL) {
                        exitcode = 1;
                }
                if (buffer == NULL) {
                        exitcode = 1;

-                       printf("create_zip_file: can not allocate %i bytes\n", buffer_size);
+                       printf("create_zip_file: can not allocate %i bytes\n", (int) buffer_size);

                        break;
                }
 
                        break;
                }
 


@@ -979,7 +970,7 @@ int main(int argc, char *argv[]) {
                        if (mtd == &mtd_kernel) {
                                if (mtd->filesize < 0x00050000) {
                                        exitcode = 1;
                        if (mtd == &mtd_kernel) {
                                if (mtd->filesize < 0x00050000) {
                                        exitcode = 1;

-                                       printf("mtd %s input file %s too unrealistic small (0x%08lX)\n", mtd->name, mtd->filesize);
+                                       printf("mtd %s input file %s too unrealistic small (0x%08lX)\n", mtd->name, mtd->filename, mtd->filesize);

                                }
                        }
 
                                }
                        }