Rémi Bernon (@rbernon) commented about dlls/bluetoothapis/sdp.c:
+ case SDP_TYPE_INT: + case SDP_TYPE_UUID: + if (size_desc > SDP_SIZEDESC_16_BYTES) return ERROR_INVALID_PARAMETER; + + st = SDP_BASIC_TYPES[type][size_desc]; + if (st == SDP_ST_NONE) return ERROR_INVALID_PARAMETER; + + if (!sdp_read_specific_type( stream, stream_size, st, data, read )) + return ERROR_INVALID_PARAMETER; + + data->type = type; + data->specificType = st; + break; + case SDP_TYPE_BOOLEAN: + if (size_desc != SDP_SIZEDESC_1_BYTE) return ERROR_INVALID_PARAMETER; + if (stream_size < sizeof(BYTE)) return ERROR_INVALID_PARAMETER; A bit sad that this one lost its parens.
Fwiw, the "space in paren Wine code style" usually only applies to function decls and calls and macros, with a couple of exceptions "where it looks better". So, sizeof operator doesn't usually have spaces inside. I don't want to bikeshed about it, but please be consistent. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6402#note_82805