From 380124c7a21fc216e760e8b925d8738292b725f8 Mon Sep 17 00:00:00 2001 From: Lukas Fleischer Date: Fri, 8 Sep 2017 20:51:56 +0200 Subject: Factor out hash computation Move code to compute the hash of a data file to a separate function. Signed-off-by: Lukas Fleischer --- src/io.c | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) (limited to 'src') diff --git a/src/io.c b/src/io.c index 600993b..4d80e3f 100644 --- a/src/io.c +++ b/src/io.c @@ -426,6 +426,18 @@ unsigned io_save_keys(void) return 1; } +static int io_compute_hash(const char *path, char *buf) +{ + FILE *fp = fopen(path, "r"); + + if (!fp) + return 0; + sha1_stream(fp, buf); + fclose(fp); + + return 1; +} + static void io_merge_data(void) { char *path_apts_new, *path_todo_new; @@ -502,22 +514,17 @@ static int resolve_save_conflict(void) static int io_check_data_files_modified() { - FILE *fp; char sha1_new[SHA1_DIGESTLEN * 2 + 1]; int ret = 1; io_mutex_lock(); - if ((fp = fopen(path_apts, "r"))) { - sha1_stream(fp, sha1_new); - fclose(fp); + if (io_compute_hash(path_apts, sha1_new)) { if (strncmp(sha1_new, apts_sha1, SHA1_DIGESTLEN * 2) != 0) goto cleanup; } - if ((fp = fopen(path_todo, "r"))) { - sha1_stream(fp, sha1_new); - fclose(fp); + if (io_compute_hash(path_todo, sha1_new)) { if (strncmp(sha1_new, todo_sha1, SHA1_DIGESTLEN * 2) != 0) goto cleanup; } -- cgit v1.2.3-70-g09d2