diff options
| author | Alexandr Miloslavskiy <alexandr.miloslavskiy@syntevo.com> | 2019-11-21 20:26:22 +0100 |
|---|---|---|
| committer | Benjamin Otte <otte@redhat.com> | 2019-11-23 18:15:19 +0100 |
| commit | fd1630e049e1b9d670d1b16ab9801628f638dd86 (patch) | |
| tree | 6ac0bf185ac4c787d8614bfb19b0f759bd083b0b /testsuite | |
| parent | e3c3cbbbcb4706203fe33b2eb471123242a7b89f (diff) | |
| download | gtk+-fd1630e049e1b9d670d1b16ab9801628f638dd86.tar.gz | |
Fix crash in parse_border()
The bug was introduced in commit:
9b7640b8 by Benjamin Otte, 2012-03-26 17:24:02
styleproperty: Make _gtk_style_property_parse_value() return a CssValue
In that commit, `values` changed from `GValue*` to `GtkCssValue**`,
but one `!G_IS_VALUE (&values[8])` was left untouched. As a result,
if `border` shorthand contains anything after color, it might crash,
depending on memory layout.
New test included.
Fixes: #751
Diffstat (limited to 'testsuite')
| -rw-r--r-- | testsuite/css/parser/border-crash-3.24.css | 4 | ||||
| -rw-r--r-- | testsuite/css/parser/border-crash-3.24.errors | 1 | ||||
| -rw-r--r-- | testsuite/css/parser/border-crash-3.24.ref.css | 0 | ||||
| -rw-r--r-- | testsuite/css/parser/meson.build | 7 |
4 files changed, 10 insertions, 2 deletions
diff --git a/testsuite/css/parser/border-crash-3.24.css b/testsuite/css/parser/border-crash-3.24.css new file mode 100644 index 0000000000..971048e2a8 --- /dev/null +++ b/testsuite/css/parser/border-crash-3.24.css @@ -0,0 +1,4 @@ +* { + border: 2px solid alpha(black, 0.3) + padding: 2px; +} diff --git a/testsuite/css/parser/border-crash-3.24.errors b/testsuite/css/parser/border-crash-3.24.errors new file mode 100644 index 0000000000..c49f1d071a --- /dev/null +++ b/testsuite/css/parser/border-crash-3.24.errors @@ -0,0 +1 @@ +border-crash-3.24.css:3: error: GTK_CSS_PROVIDER_ERROR_SYNTAX diff --git a/testsuite/css/parser/border-crash-3.24.ref.css b/testsuite/css/parser/border-crash-3.24.ref.css new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/testsuite/css/parser/border-crash-3.24.ref.css diff --git a/testsuite/css/parser/meson.build b/testsuite/css/parser/meson.build index 573f0a7087..1e171aaa36 100644 --- a/testsuite/css/parser/meson.build +++ b/testsuite/css/parser/meson.build @@ -156,12 +156,15 @@ test_data = [ 'backslash-eof-is-identifier.css', 'backslash-eof-is-identifier.errors', 'backslash-eof-is-identifier.ref.css', + 'border.css', + 'border.errors', 'border-color.css', 'border-color-currentcolor.css', 'border-color-currentcolor.ref.css', 'border-color.ref.css', - 'border.css', - 'border.errors', + 'border-crash-3.24.css', + 'border-crash-3.24.errors', + 'border-crash-3.24.ref.css', 'border-image-repeat.css', 'border-image-repeat.ref.css', 'border-image-slice.css', |