firebase / firebase-ios-sdk

Firebase SDK for Apple App Development
https://firebase.google.com
Apache License 2.0
5.67k stars 1.49k forks source link

Unreleased Resource: Database #4552

Closed gopalchinnadurai closed 4 years ago

gopalchinnadurai commented 4 years ago

We have used FirebaseRemoteConfig 4.4.0 as per our need. Some VAPT points are mentioned in below picture. Kindly help us asap.

Issue Summary:

The function insertMainTableWithValues:fromSource:() in RCNConfigDBManager.m sometimes fails to release a database resource allocated by prepareSQL:() on line 427.The program can potentially fail to release a database resource.

Issue Screenshot:

Screenshot 2019-12-24 at 4 57 33 PM
google-oss-bot commented 4 years ago

I found a few problems with this issue:

morganchen12 commented 4 years ago

Looks like the line it's referring to is this early exit, which only happens if statement fails to materialize. Closing this as a non-issue since the statement cannot be released if it is nil.

If you are able to profile this section of code and share an Instruments trace demonstrating a memory leak, please comment here and we'll re-open the issue.