Retroarch: RDB files are unreadable when compiled with GCC 10
Description
The Import Content feature appears to be broken. Selecting scan by folder and file yield the same result. The GUI indicates it's stuck scanning the first file it comes across and remains at 0%, never progresses and must be restarted. Also happens when running as the root user
Expected behaviour
[What you expected to happen]
Actual behavior
[What is actually happening]
Steps to reproduce the bug
- Compile from source, git clone, ./configure, make, sudo make install
- Capture some output with retroarch --menu --verbose >> retroarch.log 2>&1
- Stripe out Vulkan and slang noise with sed.
- Attempt to import any content either file or folder style
- Remains at 0%, logs show the last it recognises a file and matches a 'disc label'
Bisect Results
Appeared to start with 1.8.7
Version/Commit
Git Version: fda235518a
- RetroArch: 1.8.8
Environment information
- OS: Fedora 32
- Compiler: gcc/g++ 10.1.1 20200507 (Red Hat 10.1.1-1)
osirase
All 55 comments
[INFO] RetroArch 1.8.8 (Git fda235518a)
[INFO] === Build =======================================
[INFO] CPU Model Name: AMD Ryzen 5 3400G with Radeon Vega Graphics
[INFO] Capabilities: MMX MMXEXT SSE SSE2 SSE3 SSSE3 SSE4 SSE4.2 AES AVX AVX2
[INFO] Built: May 26 2020
[INFO] Version: 1.8.8
[INFO] Git: fda235518a
[INFO] =================================================
[INFO] Version of libretro API: 1
[INFO] Compiled against API: 1
[INFO] SRAM will not be saved.
[INFO] Loading history file: [/home/
[INFO] Loading history file: [/home/
[INFO] Loading history file: [/home/
[INFO] Loading history file: [/home/
[INFO] Loading favorites file: [/home/
[INFO] Parsing CUE file '/home/
[INFO] Reading first data track...
[INFO] Comparing with known magic numbers...
[INFO] Found disc label 'SLES-01135'
[INFO] [config] Saved new config to "/home/
[INFO] Content ran for a total of: 00 hours, 00 minutes, 00 seconds.
[INFO] Unloading game..
[INFO] Unloading core..
[INFO] Unloading core symbols..
osirase
on 26 May 2020
do you have the databases?
hizzlekizzle
on 26 May 2020
I would think so, I've always downloaded them from the internal updater thing.
/home/
total 65M
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-r--r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-r--r--. 1
-rw-r--r--. 1
-rw-r--r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-rw-r--. 1
-rw-r--r--. 1
Most helpful comment
This is not a bug in GCC optimization. There is an odd behavior in those optimizations but i think the underlying problem is a buffer overflow in RetroArch code.
GCC10 in -O3 and -O2 mode removes the following conditional statement
if (strncmp(header.magic_number, MAGIC_NUMBER, sizeof(MAGIC_NUMBER)) != 0)in libretro-db/libretrodb.c.
The generated assembly code do the comparison and then directly jump to the error label regardless of the value.
The suspected bufferoverflow is in struct libretrodb_header declaration.
typedef struct libretrodb_header { char magic_number[sizeof(MAGIC_NUMBER)-1]; uint64_t metadata_offset; } libretrodb_header_t;IMHO should be
typedef struct libretrodb_header { char magic_number[sizeof(MAGIC_NUMBER)]; uint64_t metadata_offset; } libretrodb_header_t;The fix is really small but i can do PR if you prefer.