Actions
Bug #49167
opens3select compilation warnings
% Done:
0%
Source:
Tags:
s3select
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
from gcc:
In file included from src/s3select/include/s3select.h:10, from src/rgw/rgw_rest_s3.cc:21: src/s3select/include/s3select_oper.h: In member function 'virtual s3selectEngine::value& s3selectEngine::logical_operand::eval_internal()': src/s3select/include/s3select_oper.h:594:21: warning: 'res' may be used uninitialized in this function [-Wmaybe-uninitialized] this->__val.num = (int64_t)b; ~~~~~~~~~~~~~~~~^~~~~~~~~~~~ src/s3select/include/s3select_oper.h:1459:10: note: 'res' was declared here bool res; ^~~
from clang:
In file included from src/rgw/rgw_rest_s3.cc:21: In file included from src/s3select/include/s3select.h:11: src/s3select/include/s3select_functions.h:541:37: warning: implicit conversion from 'long' to 'double' changes value from 9223372036854775807 to 9223372036 854775808 [-Wimplicit-const-int-float-conversion] if ((errno == ERANGE && (d == LONG_MAX || d == LONG_MIN)) || (errno != 0 && d == 0)) { ~~ ^~~~~~~~ /usr/lib64/clang/11.0.0/include/limits.h:47:19: note: expanded from macro 'LONG_MAX' #define LONG_MAX __LONG_MAX__ ^~~~~~~~~~~~ <built-in>:94:22: note: expanded from here #define __LONG_MAX__ 9223372036854775807L ^~~~~~~~~~~~~~~~~~~~ 1 warning generated.
In file included from ../src/rgw/rgw_rest_s3.cc:21: In file included from ../src/s3select/include/s3select.h:9: ../src/s3select/include/s3select_oper.h:183:57: warning: unused variable 'iter' [-Wunused-variable] std::vector<std::pair<std::string, int >>::iterator iter; ^
Actions