status-im/instabug-reactnative
2
0
Fork 0
mirror of https://github.com/status-im/instabug-reactnative.git synced 2025-03-02 14:10:40 +00:00
Code Issues Projects Releases Wiki Activity

Fix bad access to null variadic args

Browse Source 
This removes the need to access `va_list` in a non-variadic function.

By adding a variadic logging function, we can still bridge `RCTLogFunction`, without changing the public API of `IBGNSLogWithLevel` to not require a `va_list`.

This also removes the need to call the log function on the main queue, which was applied in #76 (7cc7e90a5a49629519706942334c9e6e9957e628).

Fixes #82
This commit is contained in:
Eli Perkins 2017-10-09 15:28:10 -04:00 committed by GitHub
parent 58ce2587ef
commit 741afec07b
1 changed files with 29 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
ios/RNInstabug/InstabugReactBridge.m
View File

@ -442,6 +442,15 @@ RCT_EXPORT_METHOD(isRunningLive:(RCTResponseSenderBlock)callback) {
return [iOSVersion compare:[UIDevice currentDevice].systemVersion options:NSNumericSearch] == NSOrderedDescending;
};
// Note: This function is used to bridge IBGNSLog with RCTLogFunction.
// This log function should not be used externally and is only an implementation detail.
void RNIBGLog(IBGLogLevel logLevel, NSString *format, ...) {
va_list arg_list;
va_start(arg_list, format);
IBGNSLogWithLevel(format, arg_list, logLevel);
va_end(arg_list);
}
RCTLogFunction InstabugReactLogFunction = ^(
RCTLogLevel level,
__unused RCTLogSource source,
@ -450,30 +459,27 @@ RCTLogFunction InstabugReactLogFunction = ^(
NSString *message
)
{
NSString *formatString = @"Instabug - REACT LOG: %@";
NSString *log = RCTFormatLog([NSDate date], level, fileName, lineNumber, message);
NSString *compeleteLog = [NSString stringWithFormat:@"Instabug - REACT LOG: %@", log];
dispatch_async(dispatch_get_main_queue(), ^{
va_list arg_list;
switch(level) {
case RCTLogLevelTrace:
IBGNSLogWithLevel(compeleteLog, arg_list, IBGLogLevelTrace);
break;
case RCTLogLevelInfo:
IBGNSLogWithLevel(compeleteLog, arg_list, IBGLogLevelInfo);
break;
case RCTLogLevelWarning:
IBGNSLogWithLevel(compeleteLog, arg_list, IBGLogLevelWarning);
break;
case RCTLogLevelError:
IBGNSLogWithLevel(compeleteLog, arg_list, IBGLogLevelError);
break;
case RCTLogLevelFatal:
IBGNSLogWithLevel(compeleteLog, arg_list, IBGLogLevelFatal);
break;
}
});
switch(level) {
case RCTLogLevelTrace:
RNIBGLog(IBGLogLevelTrace, formatString, log);
break;
case RCTLogLevelInfo:
RNIBGLog(IBGLogLevelInfo, formatString, log);
break;
case RCTLogLevelWarning:
RNIBGLog(IBGLogLevelWarning, formatString, log);
break;
case RCTLogLevelError:
RNIBGLog(IBGLogLevelError, formatString, log);
break;
case RCTLogLevelFatal:
RNIBGLog(IBGLogLevelFatal, formatString, log);
break;
}
};
@end