Changeset 171cc28


Ignore:
Timestamp:
04/21/07 14:21:13 (15 years ago)
Author:
Perry Lorier <perry@…>
Branches:
4.0.1-hotfixes, cachetimestamps, develop, dpdk-ndag, etsilive, getfragoff, help, libtrace4, master, ndag_format, pfring, rc-4.0.1, rc-4.0.2, rc-4.0.3, rc-4.0.4, ringdecrementfix, ringperformance, ringtimestampfixes
Children:
7be3cc4
Parents:
b916afc
Message:

Improve error handling of tools w.r.t bad output formats.
Fixes #244

Location:
tools/tracesplit
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • tools/tracesplit/tracesplit.c

    r8d504c1 r171cc28  
    126126        input=trace_create(argv[optind]);
    127127        if (trace_is_err(input)) {
    128                 trace_perror(input,"");
     128                trace_perror(input,"%s",argv[optind]);
    129129                return 1;
    130130        }
    131131
    132132        if (trace_start(input)==-1) {
    133                 trace_perror(input,"");
     133                trace_perror(input,"%s",argv[optind]);
    134134                return 1;
    135135        }
    … …  
    198198                        }
    199199                        output=trace_create_output(buffer);
     200                        if (trace_is_err_output(output)) {
     201                                trace_perror_output(output,"%s",buffer);
     202                                free(buffer);
     203                                break;
     204                        }
    200205                        trace_start_output(output);
     206                        if (trace_is_err_output(output)) {
     207                                trace_perror_output(output,"%s",buffer);
     208                                free(buffer);
     209                                break;
     210                        }
    201211                        free(buffer);
    202212                        filescreated ++;

  • tools/tracesplit/tracesplit_dir.c

    ra7282fb r171cc28  
    88        output = trace_create_output(uri);
    99        if (trace_is_err_output(output)) {
    10                 trace_err = trace_get_err_output(output);
    11                 printf("Problem creating output trace (uri=%s): %s\n",
    12                                 uri, trace_err.problem);
     10                trace_perror_output(output,"%s",uri);
     11                trace_destroy_output(output);
    1312                return NULL;
    1413        }
    1514        /* Default values for now */
    1615        trace_start_output(output);
     16        if (trace_is_err_output(output)) {
     17                trace_perror_output(output,"%s",uri);
     18                trace_destroy_output(output);
     19                return NULL;
     20        }
    1721        return output;
    1822}
    … …  
    2832        libtrace_err_t trace_err;
    2933        struct libtrace_packet_t *packet = trace_create_packet();
    30 
    3134       
    3235        if (argc < 3) {
    … …  
    4245        }
    4346        if (trace_start(input)==-1) {
    44                 trace_perror(input,"");
     47                trace_perror(input,argv[1]);
    4548                return 1;
    4649        }
    … …  
    5457                                if (!out_write) {
    5558                                        out_write = create_output(argv[3]);
    56                                         if (trace_is_err_output(out_write)) {
    57                                                 trace_perror_output(out_write,"");
     59                                        if (!out_write)
    5860                                                return 1;
    59                                         }
    6061                                }
    6162                                if (trace_write_packet(out_write, packet)==-1){
    … …  
    6667                        case 1:
    6768                                if (!in_write) {
    68                                         in_write = create_output(argv[2]);
    69                                         if (trace_is_err_output(in_write)) {
    70                                                 trace_perror_output(in_write,"");
    71                                                 return 1;
    72                                         }
    7369                                        in_write = create_output(argv[2]);
    7470                                        if (!in_write)
Note: See TracChangeset for help on using the changeset viewer.