Projects
openEuler:Mainline
SDL2
_service:tar_scm:CVE-2020-14409_CVE-2020-14410....
Sign Up
Log In
Username
Password
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File _service:tar_scm:CVE-2020-14409_CVE-2020-14410.patch of Package SDL2
From a7ff6e96155f550a5597621ebeddd03c98aa9294 Mon Sep 17 00:00:00 2001 From: Sam Lantinga <slouken@libsdl.org> Date: Wed, 17 Jun 2020 08:44:45 -0700 Subject: [PATCH] Fixed overflow in surface pitch calculation --- src/video/SDL_surface.c | 23 +++++++++++++++-------- 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/src/video/SDL_surface.c b/src/video/SDL_surface.c index 085d9ff1e17..bff826f7cc6 100644 --- a/src/video/SDL_surface.c +++ b/src/video/SDL_surface.c @@ -28,24 +28,23 @@ #include "SDL_yuv_c.h" -/* Check to make sure we can safely check multiplication of surface w and pitch and it won't overflow size_t */ -SDL_COMPILE_TIME_ASSERT(surface_size_assumptions, - sizeof(int) == sizeof(Sint32) && sizeof(size_t) >= sizeof(Sint32)); +/* Check to make sure we can safely check multiplication of surface w and pitch and it won't overflow Sint64 */ +SDL_COMPILE_TIME_ASSERT(surface_size_assumptions, sizeof(int) == sizeof(Sint32)); /* Public routines */ /* * Calculate the pad-aligned scanline width of a surface */ -static int +static Sint64 SDL_CalculatePitch(Uint32 format, int width) { - int pitch; + Sint64 pitch; if (SDL_ISPIXELFORMAT_FOURCC(format) || SDL_BITSPERPIXEL(format) >= 8) { - pitch = (width * SDL_BYTESPERPIXEL(format)); + pitch = ((Sint64)width * SDL_BYTESPERPIXEL(format)); } else { - pitch = ((width * SDL_BITSPERPIXEL(format)) + 7) / 8; + pitch = (((Sint64)width * SDL_BITSPERPIXEL(format)) + 7) / 8; } pitch = (pitch + 3) & ~3; /* 4-byte aligning for speed */ return pitch; @@ -59,11 +58,19 @@ SDL_Surface * SDL_CreateRGBSurfaceWithFormat(Uint32 flags, int width, int height, int depth, Uint32 format) { + Sint64 pitch; SDL_Surface *surface; /* The flags are no longer used, make the compiler happy */ (void)flags; + pitch = SDL_CalculatePitch(format, width); + if (pitch < 0 || pitch > SDL_MAX_SINT32) { + /* Overflow... */ + SDL_OutOfMemory(); + return NULL; + } + /* Allocate the surface */ surface = (SDL_Surface *) SDL_calloc(1, sizeof(*surface)); if (surface == NULL) { @@ -78,7 +85,7 @@ SDL_CreateRGBSurfaceWithFormat(Uint32 flags, int width, int height, int depth, } surface->w = width; surface->h = height; - surface->pitch = SDL_CalculatePitch(format, width); + surface->pitch = (int)pitch; SDL_SetClipRect(surface, NULL); if (SDL_ISPIXELFORMAT_INDEXED(surface->format->format)) {
Locations
Projects
Search
Status Monitor
Help
Open Build Service
OBS Manuals
API Documentation
OBS Portal
Reporting a Bug
Contact
Mailing List
Forums
Chat (IRC)
Twitter
Open Build Service (OBS)
is an
openSUSE project
.
浙ICP备2022010568号-2