diff options
Diffstat (limited to 'results/classifier/105/mistranslation/1326533')
| -rw-r--r-- | results/classifier/105/mistranslation/1326533 | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/results/classifier/105/mistranslation/1326533 b/results/classifier/105/mistranslation/1326533 new file mode 100644 index 00000000..1894f21e --- /dev/null +++ b/results/classifier/105/mistranslation/1326533 @@ -0,0 +1,36 @@ +mistranslation: 0.867 +device: 0.680 +instruction: 0.673 +graphic: 0.582 +semantic: 0.454 +network: 0.417 +socket: 0.325 +vnc: 0.296 +boot: 0.104 +other: 0.079 +KVM: 0.073 +assembly: 0.050 + +SDL2 UI sends a NULL to sdl_grab_start if fullscreen, which crashes + +in ui/sdl2.c: + + if (full_screen) { + gui_fullscreen = 1; + sdl_grab_start(0); + } + +Is sent, but no null checks are made in sdl_grab_start (its assumed to be an allocated pointer). So a crash happens if you start qemu -full-screen. + +It should at lease send the first [0] of the newly allocated sdl2_console through. + +Quickly looking around should look something like: + + if (full_screen) { + gui_fullscreen = 1; + sdl_grab_start(&sdl2_console[0]); + } + +The NULL pointer check has been added here: +http://git.qemu.org/?p=qemu.git;a=commitdiff;h=f2335791fd0ceb2f9e3 + |