[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 19/21] tests: Fixes test-io-channel-file by mask only owne
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 19/21] tests: Fixes test-io-channel-file by mask only owner file state mask bits |
Date: |
Wed, 9 Sep 2020 14:10:09 +0100 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Wed, Sep 09, 2020 at 05:46:15PM +0800, Yonggang Luo wrote:
> This is the error on msys2/mingw
> Running test test-io-channel-file
> **
> ERROR:../tests/test-io-channel-file.c:59:test_io_channel_file_helper:
> assertion failed (TEST_MASK & ~mask == st.st_mode & 0777): (384 == 438)
> ERROR test-io-channel-file - Bail out!
> ERROR:../tests/test-io-channel-file.c:59:test_io_channel_file_helper:
> assertion failed (TEST_MASK & ~mask == st.st_mode & 0777): (384 == 438)
>
> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> ---
> tests/test-io-channel-file.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/tests/test-io-channel-file.c b/tests/test-io-channel-file.c
> index bac2b07562..75aea6450a 100644
> --- a/tests/test-io-channel-file.c
> +++ b/tests/test-io-channel-file.c
> @@ -56,7 +56,9 @@ static void test_io_channel_file_helper(int flags)
> umask(mask);
> ret = stat(TEST_FILE, &st);
> g_assert_cmpint(ret, >, -1);
> - g_assert_cmpuint(TEST_MASK & ~mask, ==, st.st_mode & 0777);
> + /* On Windows the stat() function in the C library checks only
> + the FAT-style READONLY attribute and does not look at the ACL at all. */
> + g_assert_cmpuint(TEST_MASK & ~mask, ==, st.st_mode & 0700);
I think we will want the stronger check on non-Win32, so better to
ifdef this to use 0700 only on Win32.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH v2 14/21] cirrus: Building freebsd in a single short, (continued)
[PATCH v2 15/21] tests: Convert g_free to g_autofree macro in test-logging.c, Yonggang Luo, 2020/09/09
[PATCH v2 16/21] rcu: add uninit destructor for rcu, Yonggang Luo, 2020/09/09
[PATCH v2 17/21] tests: Fixes test-io-channel-socket.c tests under msys2/mingw, Yonggang Luo, 2020/09/09
[PATCH v2 18/21] tests: fixes aio-win32 about aio_remove_fd_handler, get it consistence with aio-posix.c, Yonggang Luo, 2020/09/09
[PATCH v2 19/21] tests: Fixes test-io-channel-file by mask only owner file state mask bits, Yonggang Luo, 2020/09/09
- Re: [PATCH v2 19/21] tests: Fixes test-io-channel-file by mask only owner file state mask bits,
Daniel P . Berrangé <=
[PATCH v2 20/21] tests: fix test-util-sockets.c, Yonggang Luo, 2020/09/09
[PATCH v2 21/21] tests: Fixes test-qdev-global-props.c, Yonggang Luo, 2020/09/09