Skip to content

Commit

Permalink
openssh: drop rejected patch fixed in 8.6p1 release
Browse files Browse the repository at this point in the history
The rationale [1] is that C11 6.5.6.9 says:
"""
When two pointers are subtracted, both shall point to elements of the
same array object, or one past the last element of the array object; the
result is the difference of the subscripts of the two array elements.
"""

In these cases the objects are arrays of char so the result is defined,
and we believe that the compiler incorrectly trapping on defined behaviour.

I also found https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63303
("Pointer subtraction is broken when using -fsanitize=undefined") which seems to support this position.

[1] https://bugzilla.mindrot.org/show_bug.cgi?id=2608

(From OE-Core rev: bb97d6ce29dbe59ac8f7f684acb6a14370d24329)

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Richard Purdie <[email protected]>
  • Loading branch information
quaresmajose authored and rpurdie committed Jul 17, 2024
1 parent 64c974e commit c0e8018
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 112 deletions.

This file was deleted.

1 change: 0 additions & 1 deletion meta/recipes-connectivity/openssh/openssh_9.7p1.bb
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ SRC_URI = "http://ftp.openbsd.org/pub/OpenBSD/OpenSSH/portable/openssh-${PV}.tar
file://sshdgenkeys.service \
file://volatiles.99_sshd \
file://run-ptest \
file://fix-potential-signed-overflow-in-pointer-arithmatic.patch \
file://sshd_check_keys \
file://0001-regress-banner.sh-log-input-and-output-files-on-erro.patch \
file://0001-systemd-Add-optional-support-for-systemd-sd_notify.patch \
Expand Down

0 comments on commit c0e8018

Please sign in to comment.