[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1631598 - in /subversion/trunk/subversion: libsvn_fs_fs/verify.c tests/libsvn_fs_fs/fs-fs-fuzzy-test.c

From: Ben Reser <ben_at_reser.org>
Date: Tue, 13 Jan 2015 11:34:55 -0800

On 1/13/15 11:13 AM, Branko Čibej wrote:
> Since it is a test, what's wrong with just casting the first vararg to
> (apr_uint64_t) instead, since we "know" (i.e., hope) that off_t won't
> overflow 64 bits ...

If you cast it to apr_uint64_t it can read into memory it shouldn't be (i.e.
the test may crash on platforms without 64-bit off_t). Maybe we don't care
about those platforms, but I'm not aware of us requiring LFS/64-bit off_t.
Received on 2015-01-13 20:35:17 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.