webpmux: fix exit status on numeric value parse error

in most cases 'ok' is set via a goto macro

Change-Id: I17c832446bf3e716d3bcd323dbcc72bec544029c
This commit is contained in:
James Zern 2014-09-11 23:36:12 -07:00
parent 2172cb626a
commit f89e1690df

View File

@ -874,7 +874,7 @@ static int GetFrameFragment(const WebPMux* mux,
ErrGet: ErrGet:
WebPDataClear(&info.bitstream); WebPDataClear(&info.bitstream);
WebPMuxDelete(mux_single); WebPMuxDelete(mux_single);
return ok; return ok && !parse_error;
} }
// Read and process config. // Read and process config.
@ -946,7 +946,8 @@ static int Process(const WebPMuxConfig* config) {
ERROR_GOTO1("ERROR: Loop count must be in the range 0 to " ERROR_GOTO1("ERROR: Loop count must be in the range 0 to "
"65535.\n", Err2); "65535.\n", Err2);
} }
if (parse_error) goto Err2; ok = !parse_error;
if (!ok) goto Err2;
params.loop_count = (int)loop_count; params.loop_count = (int)loop_count;
break; break;
} }