Revert "Check data from getenv("LIBNFC_LOG_LEVEL") and config file"
This reverts commit 846189b62c
.
It didn't solve Coverity somplain and it broke bitfield support of log_level
This commit is contained in:
parent
846189b62c
commit
252f590d5c
3 changed files with 3 additions and 12 deletions
|
@ -125,10 +125,7 @@ conf_keyvalue_context(void *data, const char *key, const char *value)
|
||||||
} else if (strcmp(key, "allow_intrusive_scan") == 0) {
|
} else if (strcmp(key, "allow_intrusive_scan") == 0) {
|
||||||
string_as_boolean(value, &(context->allow_intrusive_scan));
|
string_as_boolean(value, &(context->allow_intrusive_scan));
|
||||||
} else if (strcmp(key, "log_level") == 0) {
|
} else if (strcmp(key, "log_level") == 0) {
|
||||||
int i = atoi(value);
|
context->log_level = atoi(value);
|
||||||
if (i < 0) i = 0;
|
|
||||||
if (i > 3) i = 3;
|
|
||||||
context->log_level = i;
|
|
||||||
} else if (strcmp(key, "device.name") == 0) {
|
} else if (strcmp(key, "device.name") == 0) {
|
||||||
if ((context->user_defined_device_count == 0) || strcmp(context->user_defined_devices[context->user_defined_device_count - 1].name, "") != 0) {
|
if ((context->user_defined_device_count == 0) || strcmp(context->user_defined_devices[context->user_defined_device_count - 1].name, "") != 0) {
|
||||||
if (context->user_defined_device_count >= MAX_USER_DEFINED_DEVICES) {
|
if (context->user_defined_device_count >= MAX_USER_DEFINED_DEVICES) {
|
||||||
|
|
|
@ -87,10 +87,7 @@ log_put(const uint8_t group, const char *category, const uint8_t priority, const
|
||||||
log_level = 1;
|
log_level = 1;
|
||||||
#endif
|
#endif
|
||||||
} else {
|
} else {
|
||||||
int i = atoi(env_log_level);
|
log_level = atoi(env_log_level);
|
||||||
if (i < 0) i = 0;
|
|
||||||
if (i > 3) i = 3;
|
|
||||||
log_level = i;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// printf("log_level = %"PRIu32" group = %"PRIu8" priority = %"PRIu8"\n", log_level, group, priority);
|
// printf("log_level = %"PRIu32" group = %"PRIu8" priority = %"PRIu8"\n", log_level, group, priority);
|
||||||
|
|
|
@ -121,10 +121,7 @@ nfc_context_new(void)
|
||||||
// log level
|
// log level
|
||||||
envvar = getenv("LIBNFC_LOG_LEVEL");
|
envvar = getenv("LIBNFC_LOG_LEVEL");
|
||||||
if (envvar) {
|
if (envvar) {
|
||||||
int i = atoi(envvar);
|
res->log_level = atoi(envvar);
|
||||||
if (i < 0) i = 0;
|
|
||||||
if (i > 3) i = 3;
|
|
||||||
res->log_level = i;
|
|
||||||
}
|
}
|
||||||
#endif // ENVVARS
|
#endif // ENVVARS
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue