Memleak in get_scope()

[vaind]

init_scope() has been split out of get_scope() in #1248
However, for some reason (@jpnurmi do you know why? probably because you needed a clean object for the other call to init_scope()?) g_scope.client_sdk = get_client_sdk(); stayed in get_scope() and overwrites the previously set sentry_value_new_null() without decref.

[jpnurmi]

Thanks, good catch! Yeah, local scopes are empty - no OS context nor client SDK.

[supervacuus]

sentry_value_new_null() does not involve any allocation. A decref on a null value is a no-op from the pov of an allocator. There is no leak between client_sdk in init_scope() and outside in get_scope().