annotate src/typemap @ 34309:d7cfd8fb1d75 v9.1.0089

patch 9.1.0089: qsort() comparison functions should be transitive Commit: https://github.com/vim/vim/commit/e06e43766500ecb4cd1031fa16cf9cbebdb222c1 Author: Christian Brabandt <cb@256bit.org> Date: Fri Feb 9 19:39:14 2024 +0100 patch 9.1.0089: qsort() comparison functions should be transitive Problem: qsort() comparison functions should be transitive Solution: Do not subtract values, but rather use explicit comparisons Improve qsort() comparison functions There has been a recent report on qsort() causing out-of-bounds read & write in glibc for non transitive comparison functions https://www.qualys.com/2024/01/30/qsort.txt Even so the bug is in glibc's implementation of the qsort() algorithm, it's bad style to just use substraction for the comparison functions, which may cause overflow issues and as hinted at in OpenBSD's manual page for qsort(): "It is almost always an error to use subtraction to compute the return value of the comparison function." So check the qsort() comparison functions and change them to be safe. closes: #13980 Signed-off-by: Christian Brabandt <cb@256bit.org>
author Christian Brabandt <cb@256bit.org>
date Fri, 09 Feb 2024 19:45:06 +0100
parents dec4338a7da2
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
7
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
1 TYPEMAP
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
2 VIWIN T_VIOBJNOMUNGE
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
3 VIBUF T_VIOBJNOMUNGE
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
4
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
5 INPUT
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
6 T_VIOBJNOMUNGE
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
7 if (sv_isa($arg, \"${ntype}\")) {
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
8 IV tmp = SvIV((SV*)SvRV($arg));
3344
dec4338a7da2 updated for version 7.3.439
Bram Moolenaar <bram@vim.org>
parents: 7
diff changeset
9 $var = INT2PTR($type, tmp);
7
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
10 if (!tmp)
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
11 croak(\"$ntype no longer exists\");
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
12 }
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
13 else
3fc0f57ecb91 updated for version 7.0001
vimboss
parents:
diff changeset
14 croak(\"$var is not of type ${ntype}\")