fix: check for errors during the creation of bundle.

Author: AustinBenoit

analytics/integration_test/src/integration_test.cc

@@ -298,15 +298,13 @@ TEST_F(FirebaseAnalyticsTest, TestLogEventWithComplexParameters) {
 }
 
 TEST_F(FirebaseAnalyticsTest, TestSetDefaultEventParameters) {
-  const firebase::analytics::Parameter kDefaultParameters[] = {
+  const std::vector<Parameter> kDefaultParameters = {
       firebase::analytics::Parameter("default_parameter_double", 123.456),
       firebase::analytics::Parameter("default_parameter_int", 4),
       firebase::analytics::Parameter("default_parameter_str", "Hello World"),
   };
 
-  firebase::analytics::SetDefaultEventParameters(
-      kDefaultParameters,
-      sizeof(kDefaultParameters) / sizeof(kDefaultParameters[0]));
+  firebase::analytics::SetDefaultEventParameters(kDefaultParameters);
   firebase::analytics::LogEvent("default_parameter_event");
   firebase::analytics::SetDefaultEventParameters(nullptr, 0);
 }

analytics/src/analytics_android.cc

@@ -545,6 +545,12 @@ void SetDefaultEventParameters(const Parameter* parameters,
   jobject bundle =
       env->NewObject(util::bundle::GetClass(),
                      util::bundle::GetMethodId(util::bundle::kConstructor));
+  if (util::CheckAndClearJniExceptions(env) || !bundle) {
+    LogError("Failed to create bundle for SetDefaultEventParameters.");
+    if (bundle) env->DeleteLocalRef(bundle);
+    return;
+  }
+
   for (size_t i = 0; i < number_of_parameters; ++i) {
     const Parameter& parameter = parameters[i];
     if (!AddVariantToBundle(env, bundle, parameter.name, parameter.value)) {