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: content length parsing calls strtol() instead of strtoll() #5997

Merged
merged 1 commit into from Oct 22, 2015

Conversation

Projects
None yet
3 participants
@smithfarm
Contributor

smithfarm commented Sep 20, 2015

rgw: use strict_strtoll() for content length
instead of strict_strtol().

Backport: giant, firefly
Fixes: #10701

Reported-by: Axel Dunkel <ad@dunkel.de>
Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 1c25dba)

@smithfarm smithfarm self-assigned this Sep 20, 2015

@smithfarm smithfarm added this to the firefly milestone Sep 20, 2015

@smithfarm smithfarm added bug fix core rgw and removed core labels Sep 20, 2015

ghost pushed a commit that referenced this pull request Oct 20, 2015

Merge pull request #5997: rgw: content length parsing calls strtol() …
…instead of strtoll()

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost

This comment has been minimized.

ghost commented Oct 20, 2015

@yehudasa does this backport look good to merge ? It passed a run of the firefly rgw suite ( see http://tracker.ceph.com/issues/11644#note-111 ). Note that there were two valgrind issues but they are unrelated and fixed by #6325.

ghost pushed a commit that referenced this pull request Oct 21, 2015

Merge pull request #5997: rgw: content length parsing calls strtol() …
…instead of strtoll()

Reviewed-by: Loic Dachary <ldachary@redhat.com>

yehudasa added a commit that referenced this pull request Oct 22, 2015

Merge pull request #5997 from SUSE/wip-13178-firefly
rgw: content length parsing calls strtol() instead of strtoll()

@yehudasa yehudasa merged commit a960327 into ceph:firefly Oct 22, 2015

@smithfarm smithfarm deleted the SUSE:wip-13178-firefly branch Oct 22, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment