Rémi Bernon (@rbernon) commented about dlls/win32u/sysparams.c:
break;
- /* AMD */
- case 0x1002:
strcpy( buffer, "31.0.21921.1000" );
break;
- /* Nvidia */
- case 0x10de:
strcpy( buffer, "32.0.15.6094" );
break;
- /* Default value for any other vendor. */
- default:
strcpy( buffer, "31.0.10.1000" );
break;
- }
- return buffer;
```suggestion:-21+0 switch (vendor) { case 0x8086 /* Intel */: return "32.0.101.6314"; case 0x1002 /* AMD */: return "31.0.21921.1000"; case 0x10de /* Nvidia */: return "32.0.15.6094"; default: return "31.0.10.1000"; } ```
Doesn't seem very useful to have a buffer to return static strings.