Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rgw/sfs: segfault when running 's3tests_boto3.functional.test_s3.test_post_object_*' #191

Closed
jecluis opened this issue Oct 27, 2022 · 0 comments · Fixed by aquarist-labs/ceph#60
Assignees
Labels
area/rgw-sfs RGW & SFS related kind/bug Something isn't working
Milestone

Comments

@jecluis
Copy link
Contributor

jecluis commented Oct 27, 2022

Reproducible with
nosetests -v -s <TEST>

with test being any of the below:

 s3tests_boto3.functional.test_s3:test_post_object_set_success_code
 s3tests_boto3.functional.test_s3:test_post_object_set_invalid_success_code
 s3tests_boto3.functional.test_s3:test_post_object_upload_larger_than_chunk

Really, any test from s3tests_boto3.functional.test_s3:test_post_object_* is expected to fail.

     0> 2022-10-27T09:02:44.905+0000 7f8e6eaf46c0 -1 *** Caught signal (Segmentation fault) **
 in thread 7f8e6eaf46c0 thread_name:radosgw

 ceph version 3gw-20220804-71-g771bbfc8cee (771bbfc8cee56915e58dd2af86b57ae29da7bc45) quincy (dev)
 1: /lib64/libc.so.6(+0x3d910) [0x7f8ea9120910]
 2: (rgw::sal::SFSAtomicWriter::complete(unsigned long, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<1l, 1000000000l> > >*, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<1l, 1000000000l> > >, std::map<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, ceph::buffer::v15_2_0::list, std::less<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, std::allocator<std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const, ceph::buffer::v15_2_0::list> > >&, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<1l, 1000000000l> > >, char const*, char const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const*, rgw_zone_set*, bool*, optional_yield)+0x5bb) [0x7f8eabe0ed9b]
 3: (RGWPostObj::execute(optional_yield)+0x14a9) [0x7f8eab918c09]
 4: (rgw_process_authenticated(RGWHandler_REST*, RGWOp*&, RGWRequest*, req_state*, optional_yield, rgw::sal::Store*, bool)+0xd0a) [0x7f8eab49679a]
 5: (process_request(rgw::sal::Store*, RGWREST*, RGWRequest*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, rgw::auth::StrategyRegistry const&, RGWRestfulIO*, OpsLogSink*, optional_yield, rgw::dmclock::Scheduler*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >*, std::chrono::duration<unsigned long, std::ratio<1l, 1000000000l> >*, std::shared_ptr<RateLimiter>, int*, rgw::lua::Background*)+0x2414) [0x7f8eab499554]
 6: /lib64/libradosgw.so.2(+0x8fe28c) [0x7f8eab3ef28c]
 7: /lib64/libradosgw.so.2(+0x8fef37) [0x7f8eab3eff37]
 8: make_fcontext()
 NOTE: a copy of the executable, or `objdump -rdS <executable>` is needed to interpret this.
@jecluis jecluis added kind/bug Something isn't working area/rgw-sfs RGW & SFS related labels Oct 27, 2022
@jecluis jecluis added this to S3GW Oct 27, 2022
@jecluis jecluis moved this to Backlog in S3GW Oct 27, 2022
@jecluis jecluis changed the title rgw/sfs: segfault when running 's3tests_boto3.functional.test_s3.test_post_object_set_success_code' rgw/sfs: segfault when running 's3tests_boto3.functional.test_s3.test_post_object_set_*_code' Oct 27, 2022
@jecluis jecluis changed the title rgw/sfs: segfault when running 's3tests_boto3.functional.test_s3.test_post_object_set_*_code' rgw/sfs: segfault when running 's3tests_boto3.functional.test_s3.test_post_object_*' Oct 27, 2022
@0xavi0 0xavi0 self-assigned this Oct 28, 2022
0xavi0 referenced this issue in 0xavi0/ceph Oct 28, 2022
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
0xavi0 referenced this issue in 0xavi0/ceph Oct 28, 2022
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
0xavi0 referenced this issue in 0xavi0/ceph Oct 28, 2022
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
0xavi0 referenced this issue in 0xavi0/ceph Oct 28, 2022
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
Repository owner moved this from Backlog to Done ✅ in S3GW Oct 31, 2022
jecluis referenced this issue in jecluis/ceph Nov 11, 2022
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
jecluis referenced this issue in aquarist-labs/ceph Dec 14, 2022
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
jecluis referenced this issue in aquarist-labs/ceph Jan 15, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
jecluis referenced this issue in aquarist-labs/ceph Jan 24, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
jecluis referenced this issue in aquarist-labs/ceph Jan 24, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
m-ildefons pushed a commit to m-ildefons/s3gw that referenced this issue Feb 6, 2023
@jhmarina jhmarina added this to the v0.8.0 milestone Mar 7, 2023
irq0 referenced this issue in irq0/ceph Mar 13, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
irq0 referenced this issue in irq0/ceph Mar 15, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
irq0 referenced this issue in irq0/ceph Mar 15, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
irq0 referenced this issue in irq0/ceph Mar 15, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
irq0 referenced this issue in irq0/ceph Mar 17, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
irq0 referenced this issue in aquarist-labs/ceph Mar 17, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
irq0 referenced this issue in aquarist-labs/ceph Mar 17, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
0xavi0 referenced this issue in aquarist-labs/ceph Oct 5, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
0xavi0 referenced this issue in 0xavi0/ceph Oct 5, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
0xavi0 referenced this issue in aquarist-labs/ceph Oct 18, 2023
`mtime` parameter was logged but that's an output variable and might be
`nullptr`.

Avoid logging it and check if it's `nullptr` before setting the value.

Fixes: aquarist-labs/s3gw-tools#191
Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rgw-sfs RGW & SFS related kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants