Fixes for gdk_memory_texture_new_subtexture checks
authorBenjamin Otte <otte@redhat.com>
Mon, 13 Feb 2023 00:08:53 +0000 (01:08 +0100)
committerMatthias Clasen <mclasen@redhat.com>
Fri, 3 Mar 2023 17:24:46 +0000 (11:24 -0600)
There were several mistakes here.

gdk/gdkmemorytexture.c

index 641386b786fb40f7327a54fef0b31aa75c85a46f..85d9ba0a259f5d3882db6d8629d7ad2939f6103a 100644 (file)
@@ -178,10 +178,10 @@ gdk_memory_texture_new_subtexture (GdkMemoryTexture  *source,
   GBytes *bytes;
 
   g_return_val_if_fail (GDK_IS_MEMORY_TEXTURE (source), NULL);
-  g_return_val_if_fail (x >= 0 || x < GDK_TEXTURE (source)->width, NULL);
-  g_return_val_if_fail (y >= 0 || y < GDK_TEXTURE (source)->height, NULL);
-  g_return_val_if_fail (width > 0 || x + width <= GDK_TEXTURE (source)->width, NULL);
-  g_return_val_if_fail (height > 0 || y + height <= GDK_TEXTURE (source)->height, NULL);
+  g_return_val_if_fail (x >= 0 && x < GDK_TEXTURE (source)->width, NULL);
+  g_return_val_if_fail (y >= 0 && y < GDK_TEXTURE (source)->height, NULL);
+  g_return_val_if_fail (width > 0 && x + width <= GDK_TEXTURE (source)->width, NULL);
+  g_return_val_if_fail (height > 0 && y + height <= GDK_TEXTURE (source)->height, NULL);
 
   texture = GDK_TEXTURE (source);
   bpp = gdk_memory_format_bytes_per_pixel (texture->format);