Skip to content

gh-129107: two cases where second operand needs lock #130227

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.

Sign up for GitHub

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

Merged
kumaraditya303 merged 9 commits into from
Feb 27, 2025
Merged

gh-129107: two cases where second operand needs lock #130227

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7e7bef5
gh-129107: two cases where second operand needs lock
tom-pytel Feb 17, 2025
3c62f81
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 17, 2025
d36b4aa
lock second operand in bytearray_iconcat as well
tom-pytel Feb 17, 2025
01f6c98
Merge branch 'main' into fix-issue-129107b
tom-pytel Feb 17, 2025
2f92311
remove unnecessary guard
tom-pytel Feb 23, 2025
fe15cf3
bytearray_ass_subscript use PyByteArray_Check()
tom-pytel Feb 24, 2025
9b06d82
bytearray_mod conditional lock second arg
tom-pytel Feb 26, 2025
c3236ca
Merge branch 'main' into fix-issue-129107b
tom-pytel Feb 26, 2025
afa6cec
Merge branch 'main' into fix-issue-129107b
tom-pytel Feb 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
bytearray_ass_subscript use PyByteArray_Check()
  • Loading branch information
@tom-pytel
tom-pytel committed Feb 24, 2025
commit fe15cf3913f0180b0bd0bb2ec99aaaff75fd3646
10 changes: 3 additions & 7 deletions Objects/bytearrayobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,6 @@ static PyObject *
bytearray_iconcat_lock_held(PyObject *op, PyObject *other)
{
_Py_CRITICAL_SECTION_ASSERT_OBJECT_LOCKED(op);
_Py_CRITICAL_SECTION_ASSERT_OBJECT_LOCKED(other);
PyByteArrayObject *self = _PyByteArray_CAST(op);

Py_buffer vo;
Expand Down Expand Up @@ -359,9 +358,9 @@ static PyObject *
bytearray_iconcat(PyObject *op, PyObject *other)
{
PyObject *ret;
Py_BEGIN_CRITICAL_SECTION2(op, other);
Py_BEGIN_CRITICAL_SECTION(op);
ret = bytearray_iconcat_lock_held(op, other);
Py_END_CRITICAL_SECTION2();
Py_END_CRITICAL_SECTION();
return ret;
}

Expand Down Expand Up @@ -708,9 +707,6 @@ static int
bytearray_ass_subscript_lock_held(PyObject *op, PyObject *index, PyObject *values)
{
_Py_CRITICAL_SECTION_ASSERT_OBJECT_LOCKED(op);
if (values != NULL) {
_Py_CRITICAL_SECTION_ASSERT_OBJECT_LOCKED(values);
}
PyByteArrayObject *self = _PyByteArray_CAST(op);
Py_ssize_t start, stop, step, slicelen;
char *buf = PyByteArray_AS_STRING(self);
Expand Down Expand Up @@ -868,7 +864,7 @@ static int
bytearray_ass_subscript(PyObject *op, PyObject *index, PyObject *values)
{
int ret;
if (values != NULL) {
if (values != NULL && PyByteArray_Check(values)) {
Py_BEGIN_CRITICAL_SECTION2(op, values);
ret = bytearray_ass_subscript_lock_held(op, index, values);
Py_END_CRITICAL_SECTION2();
Expand Down