We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

reproducible code ver2

sdl2_UpdateWindowSurface_failed_ver2.c (text/plain), 3.80 KB, created by Koichi Yoshida on 2019-07-28 05:07:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Koichi Yoshida

Created: 2019-07-28 05:07:14 UTC

Size: 3.80 KB

patch

obsolete

>ï»¿#include <stdint.h>
>#include <stdbool.h>
>#include <stdio.h>
>#include <assert.h>
>#include <sys/stat.h>
>#include <string.h>
>#include <unistd.h>
>#include <SDL2/SDL.h>
>
>/*
>	This is reproducible code for the recieving SEGV when drawing to resized window surface.
>
>	Tested environment:
>		debian 10.0.0 64bit
>		GNOME
>		sdl2.0.9+dfsg1-1
>*/
>
>
>/* Please choose from 0 to 3. */
>#define TEST_CASE	0
>
>
>#if TEST_CASE == 0
>	/* It dows not work. It will get SEGV at draw pixels. */
>	#define SLEEP_AFTER_RESIZE_WINDOW			0
>	#define CHECK_IF_RESIZE_WINDOW_SUCCEEDED	0
>#endif
>
>#if TEST_CASE == 1
>	/* It works. */
>	#define SLEEP_AFTER_RESIZE_WINDOW			0
>	#define CHECK_IF_RESIZE_WINDOW_SUCCEEDED	1
>#endif
>
>#if TEST_CASE == 2
>	/* It works. */
>	#define SLEEP_AFTER_RESIZE_WINDOW			1
>	#define CHECK_IF_RESIZE_WINDOW_SUCCEEDED	0
>#endif
>
>#if TEST_CASE == 3
>	/* It works. */
>	#define SLEEP_AFTER_RESIZE_WINDOW			1
>	#define CHECK_IF_RESIZE_WINDOW_SUCCEEDED	1
>#endif
>
>
>int
>main()
>{
>	/* Initialize SDL */
>	if (SDL_Init(SDL_INIT_EVERYTHING) != 0) {
>		printf("SDL_Init(SDL_INIT_EVERYTHING) failed: %s", SDL_GetError()); fflush(stdout);
>		exit(EXIT_FAILURE);
>	}
>
>	/* Create a main window */
>	#define WINDOW_WIDTH	512
>	#define WINDOW_HEIGHT	512
>	SDL_Window *const window = SDL_CreateWindow(
>		"test",
>		SDL_WINDOWPOS_UNDEFINED,
>		SDL_WINDOWPOS_UNDEFINED,
>#if 1
>		640, 480,
>#else
>		WINDOW_WIDTH, WINDOW_HEIGHT,
>#endif
>		SDL_WINDOW_RESIZABLE);
>	if (window == NULL) {
>		printf("SDL_CreateWindow() failed: %s", SDL_GetError()); fflush(stdout);
>		exit(EXIT_FAILURE);
>	}
>	SDL_SetWindowSize(window, WINDOW_WIDTH, WINDOW_HEIGHT);
>#if SLEEP_AFTER_RESIZE_WINDOW
>	sleep(1);
>#endif
>
>	SDL_Surface *surface = SDL_GetWindowSurface(window);
>	if (surface == NULL) {
>		printf("SDL_GetWindowSurface() failed: %s", SDL_GetError()); fflush(stdout);
>		exit(EXIT_FAILURE);
>	}
>
>	/* Main loop */
>	bool bQuit = false;
>	for (int frame = 0; bQuit == false; frame++) {
>		SDL_Event event;
>		while (SDL_PollEvent(&event)) {
>			switch (event.type) {
>				case SDL_KEYDOWN: {
>					if (event.key.keysym.sym == SDLK_ESCAPE) {
>						bQuit = true;
>					}
>				} break;
>				case SDL_QUIT: {
>					bQuit = true;
>				} break;
>			}
>		}
>
>		/* Clear the screen */
>		if (SDL_FillRect(surface, NULL, 0) < 0) {
>			printf("SDL_FillRect failed: %s\n", SDL_GetError()); fflush(stdout);
>			exit(EXIT_FAILURE);
>		}
>
>		/* Draw something */
>#if CHECK_IF_RESIZE_WINDOW_SUCCEEDED
>		if (surface->w == WINDOW_WIDTH && surface->h == WINDOW_HEIGHT)
>#endif
>		{
>			/* Map surface */
>			if (SDL_MUSTLOCK(surface)) {
>				if (SDL_LockSurface(surface) < 0) {
>					printf("SDL_LockSurface failed.\n"); fflush(stdout);
>					exit(EXIT_FAILURE);
>				}
>			}
>
>			/* Retrieve map information */
>			printf("surface->w %d\n", surface->w); fflush(stdout);
>			printf("surface->h %d\n", surface->h); fflush(stdout);
>			printf("surface->pixels %p\n", surface->pixels); fflush(stdout);
>			printf("surface->pixels %d\n", surface->pitch); fflush(stdout);
>			void *begin = (void *)surface->pixels;
>			int pitchInBytes = surface->pitch;
>
>			/* Draw */
>			for (int y = 0; y < WINDOW_HEIGHT; y++) {
>				for (int x = 0; x < WINDOW_WIDTH; x++) {
>					uint8_t *pixel = (uint8_t *)(
>						((uintptr_t)begin) + y * pitchInBytes + x * 4
>					);
>					pixel[0] = x ^ y ^ frame;
>					pixel[1] = x ^ y ^ frame;
>					pixel[2] = x ^ y ^ frame;
>					pixel[3] = 255;
>				}
>			}
>
>			/* Unmap surface */
>			if (SDL_MUSTLOCK(surface)) {
>				SDL_UnlockSurface(surface);
>			}
>		}
>
>		/* Update the screen */
>		if (SDL_UpdateWindowSurface(window) < 0) {
>			printf("SDL_UpdateWindowSurface failed: %s\n", SDL_GetError()); fflush(stdout);
>			surface = SDL_GetWindowSurface(window);
>			if (surface == NULL) {
>				printf("SDL_GetWindowSurface() failed: %s", SDL_GetError()); fflush(stdout);
>				exit(EXIT_FAILURE);
>			}
>		}
>		SDL_Delay(10);
>	}
>
>	/* Terminate SDL */
>	SDL_DestroyWindow(window);
>	SDL_Quit();
>
>	exit(EXIT_SUCCESS);
>}
>

Actions: View

Attachments on bug 4740: 3901 | 3906

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.