AU: Verify source rootfs/kernel hashes before applying delta.

AU: Verify source rootfs/kernel hashes before applying delta.

New style full updates will not send the old rootfs hash so no check takes
place.

BUG=7562
TEST=unit tests, gmerged on device and tested with good/bad source partition

Change-Id: I65b28bf57110e4d87472d4aea59121878cde24b0

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=698d041

[adlr, 2010-10-12 19:38:07]

LGTM w/ nit

http://codereview.chromium.org/3712003/diff/4001/5004
File delta_performer.h (right):

http://codereview.chromium.org/3712003/diff/4001/5004#newcode99
delta_performer.h:99: void set_current_kernel_hash(std::vector<char>& hash) {
hm, I wouldn't have expected that a method that takes a const reference would
actually hang onto that reference. Maybe make it take a pointer?

[petkov, 2010-10-13 15:01:40]

Fixed to work with the server-side change:

http://codereview.chromium.org/3748001/show

This CL needs to go in after the server side one.

PTAL at both CLs. Thanks!

http://codereview.chromium.org/3712003/diff/4001/5004
File delta_performer.h (right):

http://codereview.chromium.org/3712003/diff/4001/5004#newcode99
delta_performer.h:99: void set_current_kernel_hash(std::vector<char>& hash) {
On 2010/10/12 19:38:07, adlr wrote:
> hm, I wouldn't have expected that a method that takes a const reference would
> actually hang onto that reference. Maybe make it take a pointer?

Done.

[adlr, 2010-10-13 17:38:35]

LGTM