On 03 Jul 2020, at 11:02, Ruediger Pluem <[email protected]> wrote: >> apr_sha1_init(&context); >> nbytes = sizeof(buf); >> - while (apr_file_read(fd, buf, &nbytes) == APR_SUCCESS) { >> + while ((status = apr_file_read(fd, buf, &nbytes)) == APR_SUCCESS) { > > Don't we need to set the fp in fd via apr_file_seek(fd, APR_CUR, to 0? > In case we did not open the fd on our own it might be somewhere in the file > and the digest would only base on a part of the file?
We do indeed - r1879469. Regards, Graham —
