Making Xorriso_prescan_args() safe against misunderstandings

xorriso/xorriso.c

@@ -7674,13 +7674,13 @@ int Xorriso_option_as(struct XorrisO *xorriso, int argc, char **argv,
 
  end_idx= Xorriso_end_idx(xorriso, argc, argv, *idx, 1);
  idx_count= end_idx-(*idx);
- if(end_idx<=0) {
+ if(end_idx<=0 || (*idx)>=argc) {
    if(idx_count<1)
      sprintf(xorriso->info_text,
-             "-as : Not enough arguments given. Needed: whom do_what");
+             "-as : Not enough arguments given. Needed: whom do_what --");
    else
      sprintf(xorriso->info_text,
-             "-as %s : Not enough arguments given. Needed: do_what",
+             "-as %s : Not enough arguments given. Needed: do_what --",
              argv[*idx]);
    Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "FAILURE", 0);
    ret= 0; goto ex;
@@ -8217,6 +8217,11 @@ int Xorriso_option_compare(struct XorrisO *xorriso, char *disk_path,
  ipth= iso_path;
  if(ipth[0]==0)
    ipth= disk_path;
+ if(disk_path[0]==0) {
+   sprintf(xorriso->info_text, "-compare: Empty disk_path given");
+   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "SORRY", 1);
+   return(0);
+ }
  ret= Xorriso_normalize_img_path(xorriso, xorriso->wdx, disk_path, eff_origin,
                                  2|4|8);
  if(ret<=0)
@@ -9715,17 +9720,24 @@ int Xorriso_option_map(struct XorrisO *xorriso, char *disk_path,
                        char *iso_path, int flag)
 {
  int ret;
- char eff_origin[SfileadrL], eff_dest[SfileadrL];
+ char eff_origin[SfileadrL], eff_dest[SfileadrL], *ipth;
 
  if(!(flag&2))
    Xorriso_pacifier_reset(xorriso, 0);
 
+ ipth= iso_path;
+ if(ipth[0]==0)
+   ipth= disk_path;
+ if(disk_path[0]==0) {
+   sprintf(xorriso->info_text, "-map: Empty disk_path given");
+   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "SORRY", 1);
+   return(0);
+ }
  ret= Xorriso_normalize_img_path(xorriso, xorriso->wdx, disk_path, eff_origin,
                                  2|4);
  if(ret<=0)
    return(ret);
- ret= Xorriso_normalize_img_path(xorriso, xorriso->wdi, iso_path, eff_dest,
+ ret= Xorriso_normalize_img_path(xorriso, xorriso->wdi, ipth, eff_dest, 2);
-                                 2);
  if(ret<=0)
    return(ret);
  ret= Xorriso_graft_in(xorriso, NULL, eff_origin, eff_dest,
@@ -9857,8 +9869,10 @@ int Xorriso_option_not_leaf(struct XorrisO *xorriso, char *pattern, int flag)
 {
  regex_t re;
  char regexpr[2*SfileadrL+2], sfe[5*SfileadrL];
- int ret;
+ int ret= 0;
 
+ if(pattern[0]==0)
+   {ret= 0; goto cannot_add;}
  Xorriso__bourne_to_reg(pattern, regexpr, 0);
  if(regcomp(&re, regexpr, 0)!=0)
    {ret= 0; goto cannot_add;}
@@ -9884,7 +9898,7 @@ int Xorriso_option_not_list(struct XorrisO *xorriso, char *adr, int flag)
  Xorriso_pacifier_reset(xorriso, 0);
  if(adr[0]==0) {
    sprintf(xorriso->info_text,"Empty file name given with -not_list");
-   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "WARNING", 0);
+   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "FAILURE", 0);
    return(0);
  }
  fp= Afile_fopen(adr,"rb",((!!xorriso->packet_output)<<6));
@@ -10796,6 +10810,12 @@ int Xorriso_option_update(struct XorrisO *xorriso, char *disk_path,
  ipth= iso_path;
  if(ipth[0]==0)
    ipth= disk_path;
+ if(disk_path[0]==0) {
+   sprintf(xorriso->info_text, "-update: Empty disk_path given");
+   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "SORRY", 1);
+   return(0);
+ }
+
  ret= Xorriso_normalize_img_path(xorriso, xorriso->wdx, disk_path, eff_origin,
                                  2|4|8);
  if(ret<=0)
@@ -11024,7 +11044,96 @@ int Xorriso_normalize_command(struct XorrisO *xorriso, char *original_cmd,
  for(dash_pt= *cmd; *dash_pt!=0; dash_pt++)
    if(*dash_pt=='-')
      *dash_pt= '_';
- return(1);
+ return(was_dashed);
+}
+
+
+/* @param flag bit0= do not warn of unknown option
+   @return <=0 error,
+           1=count is valid, 2=dashed unknown, 3=undashed unknown
+*/
+int Xorriso_count_args(struct XorrisO *xorriso, int argc, char **argv,
+                       int *count, int flag)
+{
+ int ret, was_dashed= 0, i;
+ char *cmd, cmd_data[2*SfileadrL];
+ static char arg0_commands[][40]= {
+    "ban_stdio_write","commit","commit_eject","devices","end","help",
+    "list_formats","no_rc","print_size","pwd","pwdi","pwdx",
+    "rollback","rollback_end","tell_media_space","toc","version","--",""
+ };
+ static char arg1_commands[][40]= {
+    "abort_on","add_plainly","blank","cd","cdi","cdx","close","dev",
+    "dummy","dialog","disk_pattern","eject","iso_rr_pattern","follow",
+    "format","fs","gid","history","indev","joliet","mark","not_leaf",
+    "not_list","not_mgt","options_from_file","overwrite","outdev",
+    "padding","path_list","pathspecs","pkt_output","print","prompt",
+    "prog","publisher","reassure","report_about","rom_toc_scan",
+    "session_log","speed","split_size","status","status_history_max",
+    "temp_mem_limit","uid","volid","use_readline",""
+ };
+ static char arg2_commands[][40]= {
+    "alter_date","alter_date_r","boot_image","compare","compare_r",
+    "errfile_log","error_behavior","load","logfile",
+    "map","map_single","page","return_with","update","update_r",
+    ""
+ };
+ static char arg4_commands[][40]= {
+    "cut_out",""
+ };
+ static char argn_commands[][40]= {
+    "add","as","chgrp","chgrpi","chgrp_r","chgrp_ri","chmod","chmodi",
+    "chmod_r","chmod_ri","chown","chowni","chown_r","chown_ri",
+    "cpr","cpri","du","dui","dus","dusi","dux","dusx","find","findi","findx",
+    "ls","lsi","lsl","lsli","lsd","lsdi","lsdl","lsdli",
+    "lsx","lslx","lsdx","lsdlx","mv","mvi","mkdir","mkdiri",
+    "not_paths","rm","rmi","rm_r","rm_ri","rmdir","rmdiri",
+    ""
+ };
+
+ *count= 0;
+ if(argc<=0)
+   return(-1);
+ ret= Xorriso_normalize_command(xorriso, argv[0], -1,
+                                cmd_data, sizeof(cmd_data), &cmd, 0);
+ if(ret<0)
+   return(ret);
+ was_dashed= (ret>0);
+ if(cmd[0]=='#' || cmd[0]==0) {
+   /* ignore comment line and empty option */;
+   return(1);
+ }
+ for(i=0; arg0_commands[i][0]!=0; i++)
+   if(strcmp(arg0_commands[i], cmd)==0)
+     return(1);
+ *count= 1;
+ for(i=0; arg1_commands[i][0]!=0; i++)
+   if(strcmp(arg1_commands[i], cmd)==0)
+     return(1);
+ *count= 2;
+ for(i=0; arg2_commands[i][0]!=0; i++)
+   if(strcmp(arg2_commands[i], cmd)==0)
+     return(1);
+ *count= 4;
+ for(i=0; arg4_commands[i][0]!=0; i++)
+   if(strcmp(arg4_commands[i], cmd)==0)
+     return(1);
+ *count= 0;
+ for(i=0; argn_commands[i][0]!=0; i++)
+   if(strcmp(argn_commands[i], cmd)==0) {
+     ret= Xorriso_end_idx(xorriso, argc, argv, 1, 1);
+     if(ret<1)
+       return(ret);
+     *count= ret-1;
+     return(1);
+   }
+
+ if(!(flag&1)) {
+   sprintf(xorriso->info_text, "Unknown option : '%s'", argv[0]);
+   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "WARNING", 0);
+ }
+
+ return(2+!was_dashed);
 }
 
 
@@ -11038,8 +11147,10 @@ return:
  bit0= recursion
 */
 {
- int ret, was_dashed, end_ret;
+ int ret, was_dashed, end_ret, num1, num2;
- int num1, num2;
+#ifdef NIX
+ int arg_count;
+#endif
  char *cmd, *original_cmd, cmd_data[2*SfileadrL], *arg1, *arg2;
  
  if(xorriso==NULL)
@@ -11064,22 +11175,31 @@ next_command:;
  }
  was_dashed= 0;
 
-#ifdef NIX
-
- if(cmd[0]=='-' && cmd[1]!='-' && cmd[1]!=0) {
-   was_dashed= 1;
-   cmd++;
- }
-
-#else /* NIX */
-
  ret= Xorriso_normalize_command(xorriso, original_cmd, -1,
                                 cmd_data, sizeof(cmd_data), &cmd, 0);
  if(ret<0)
    goto eval_any_problems;
  was_dashed= ret;
 
-#endif /* ! NIX */
+#ifdef NIX
+ ret= Xorriso_count_args(xorriso, argc-(*idx), argv+(*idx), &arg_count, 0);
+ if(ret==1) {
+   sprintf(xorriso->info_text, "Option '%s' : %d :", original_cmd, arg_count);
+   if(arg_count>0 && (*idx)+arg_count < argc)
+     sprintf(xorriso->info_text+strlen(xorriso->info_text), " '%s'",
+             argv[(*idx)+1]);
+   if(arg_count>1 && (*idx)+arg_count < argc)
+     sprintf(xorriso->info_text+strlen(xorriso->info_text), " '%s'",
+             argv[(*idx)+2]);
+   if(arg_count>2 && (*idx)+arg_count < argc)
+     sprintf(xorriso->info_text+strlen(xorriso->info_text), " ... '%s'",
+             argv[(*idx)+arg_count]);
+   else if(arg_count>3 && (*idx)+arg_count-1 < argc)
+     sprintf(xorriso->info_text+strlen(xorriso->info_text), " ... '%s'",
+             argv[(*idx)+arg_count-1]);
+   Xorriso_msgs_submit(xorriso, 0, xorriso->info_text, 0, "DEBUG", 0);
+ }
+#endif /* NIX */
 
  (*idx)++;
 
@@ -11492,10 +11612,6 @@ next_command:;
    (*idx)+= 2;
    ret= Xorriso_option_update(xorriso, arg1, arg2, 1|8);
 
- } else if(strcmp(cmd,"volid")==0) {
-   (*idx)++;
-   ret= Xorriso_option_volid(xorriso,arg1,0);
-
  } else if(strcmp(cmd,"use_readline")==0) {
    (*idx)++;
    ret= Xorriso_option_use_readline(xorriso, arg1, 0);
@@ -11503,6 +11619,10 @@ next_command:;
  } else if(strcmp(cmd,"version")==0){
    ret= Xorriso_option_version(xorriso, 0);
 
+ } else if(strcmp(cmd,"volid")==0) {
+   (*idx)++;
+   ret= Xorriso_option_volid(xorriso,arg1,0);
+
  } else if(strcmp(cmd,"--")==0){
    /* tis ok */;
 
@@ -11663,7 +11783,7 @@ int Xorriso_prescan_args(struct XorrisO *xorriso, int argc, char **argv,
    1  ok, go on
 */
 {
- int i, ret, was_dashed, num2;
+ int i, ret, was_dashed, num2, arg_count;
  int was_report_about= 0, was_abort_on= 0, was_return_with= 0;
  char *cmd, *original_cmd, cmd_data[5*SfileadrL], *arg1, *arg2;
 
@@ -11671,22 +11791,11 @@ int Xorriso_prescan_args(struct XorrisO *xorriso, int argc, char **argv,
    original_cmd= cmd= argv[i];
    was_dashed= 0;
 
-#ifdef NIX
-
-   if(cmd[0]=='-' && cmd[1]!='-' && cmd[1]!=0) {
-     was_dashed= 1;
-     cmd++;
-   }
-
-#else /* NIX */
-
    was_dashed= Xorriso_normalize_command(xorriso, original_cmd, i,
                                          cmd_data, sizeof(cmd_data), &cmd, 0);
    if(was_dashed<0)
      return(-1);
 
-#endif /* ! NIX */
-
    arg1= "";
    if(i+1<argc)
      arg1= argv[i+1];
@@ -11755,6 +11864,10 @@ protect_stdout:;
         strcmp(arg1, "genisofs")==0)
        goto protect_stdout;
 
+   } else {
+     ret= Xorriso_count_args(xorriso, argc-i, argv+i, &arg_count, 1);
+     if(ret==1)
+       i+= arg_count;
    }
  }
  return(1);

xorriso/xorriso_timestamp.h

@@ -1 +1 @@
-#define Xorriso_timestamP "2008.05.18.084729"
+#define Xorriso_timestamP "2008.05.20.075142"