Nikolay Sivov : amstream/tests: Correct sizeof expression to actually get array size (PVS-Studio).

Alexandre Julliard julliard at wine.codeweavers.com
Tue Oct 13 10:53:46 CDT 2015


Module: wine
Branch: master
Commit: b752d3d44b05fd6d02b03d73ef488bf85b08c3bf
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=b752d3d44b05fd6d02b03d73ef488bf85b08c3bf

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Mon Oct 12 22:15:56 2015 +0300

amstream/tests: Correct sizeof expression to actually get array size (PVS-Studio).

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/amstream/tests/amstream.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/amstream/tests/amstream.c b/dlls/amstream/tests/amstream.c
index 909a738..1fcffea 100644
--- a/dlls/amstream/tests/amstream.c
+++ b/dlls/amstream/tests/amstream.c
@@ -425,9 +425,9 @@ static void test_media_streams(void)
                 ok(!lstrcmpW(info.achName, id), "Pin name is %s instead of %s\n", wine_dbgstr_w(info.achName), wine_dbgstr_w(id));
                 hr = IPin_EnumMediaTypes(pins[i], &enum_media_types);
                 ok(hr == S_OK, "IPin_EnumMediaTypes returned: %x\n", hr);
-                hr = IEnumMediaTypes_Next(enum_media_types, sizeof(media_types) / sizeof(AM_MEDIA_TYPE), media_types, &nb_media_types);
+                hr = IEnumMediaTypes_Next(enum_media_types, sizeof(media_types) / sizeof(media_types[0]), media_types, &nb_media_types);
                 ok(SUCCEEDED(hr), "IEnumMediaTypes_Next returned: %x\n", hr);
-                ok(nb_media_types == 0, "nb_media_types should be 0 instead of %u\n", nb_media_types);
+                ok(nb_media_types > 0, "nb_media_types should be >0\n");
                 IEnumMediaTypes_Release(enum_media_types);
                 IPin_Release(pins[i]);
             }




More information about the wine-cvs mailing list