search

ericmckean / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

md5 for y4m output is incorrect #698

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
v1.3.0-835-g29c1af2

$ for v in ./vpxdec.{v1.3.0,v1.3.0-835-g29c1af2}; do \
  $v vp90-2-07-frame_parallel.webm --md5 -o out.$(basename $v).y4m; done

bc92e731cda4893387b42f63bd00f1f2  out.vpxdec.v1.3.0.y4m
6a48450207785d0be2922457f77a4bc4  out.vpxdec.v1.3.0-835-g29c1af2.y4m

$ md5sum *.y4m
bc92e731cda4893387b42f63bd00f1f2  out.vpxdec.v1.3.0-835-g29c1af2.y4m
bc92e731cda4893387b42f63bd00f1f2  out.vpxdec.v1.3.0.y4m

Though broken in its own way with y4m the issue began here:

commit cf46bc7bfe4b0a33f98b7c6aff5514886b3b6115
Author: Dmitry Kovalev <dkovalev@google.com>
Date:   Fri Jan 17 10:57:55 2014 -0800

    Removing out_put() function from vpxdec.c.

    Adding two new functions: update_image_md5() and write_image_file().

    Change-Id: Ic3e7e6ef08e70fb15ab021c92127872f29bcad9b

Original issue reported on code.google.com by jz...@google.com on 24 Jan 2014 at 11:14

GoogleCodeExporter commented 9 years ago 
Fixed in https://gerrit.chromium.org/gerrit/#/c/68536/

Original comment by dkova...@google.com on 29 Jan 2014 at 8:03