alecmuffett / certificate-transparency

Automatically exported from code.google.com/p/certificate-transparency
0 stars 0 forks source link

Monitor (C++) crashes in case of STHs with refreshed timestamps w/ patch #36

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Steps to reproduce:
1. Run the monitor loop.
2. Wait until it fetches some STH and right after that an STH with just an 
updated timestamp (i.e. same tree size).
3. Crash.

According to the RFC this is normal behaviour but a CHECK in the monitor fails 
(as it is treated as misbehaviour).

The patch can be found here:
https://codereview.appspot.com/72580043

Side note: The monitor in this patch set also logs (non-critical) network 
problems while fetching STHs.

Original issue reported on code.google.com by Oliver.W...@gmail.com on 7 Mar 2014 at 6:41