diff --git a/dlls/user32/tests/menu.c b/dlls/user32/tests/menu.c
index 878decf96b0..7382ec08e99 100644
--- a/dlls/user32/tests/menu.c
+++ b/dlls/user32/tests/menu.c
@@ -3057,7 +3057,7 @@ static void test_menu_circref(void)
     ret = ModifyMenuA( menu1, (UINT_PTR)menu2, MF_POPUP, (UINT_PTR)menu2, "menu 2");
 todo_wine
     ok( !ret ||
-            broken( 0), /* win98, NT */
+            broken( ret), /* win98, NT */
             "ModifyMenu should have failed.\n");
     if( !ret) { /* will probably stack fault if the ModifyMenu succeeded */
         ret = GetMenuState( menu1, 123, 0);