diff options
| author | Timothy Arceri <[email protected]> | 2020-03-28 02:17:54 +1100 |
|---|---|---|
| committer | Marge Bot <[email protected]> | 2020-03-31 23:43:31 +0000 |
| commit | 0f4a81430e65e09db13d2472fd46105a95ea625d (patch) | |
| tree | f871272a02a8f08b2e6cfe1e0092598ade52d383 /src/compiler | |
| parent | 31011c7a39f0d054b97b730c8928176d72c9707c (diff) | |
nir: fix crash in varying packing on interface mismatch
For example when the outputs are scalars but the inputs are struct
members.
Fixes: 26aa460940f6 ("nir: rewrite varying component packing")
Reviewed-By: Timur Kristóf <[email protected]>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4351>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4351>
Diffstat (limited to 'src/compiler')
| -rw-r--r-- | src/compiler/nir/nir_linking_helpers.c | 24 |
1 files changed, 22 insertions, 2 deletions
diff --git a/src/compiler/nir/nir_linking_helpers.c b/src/compiler/nir/nir_linking_helpers.c index 7864f042636..ec1bd77928c 100644 --- a/src/compiler/nir/nir_linking_helpers.c +++ b/src/compiler/nir/nir_linking_helpers.c @@ -569,6 +569,7 @@ gather_varying_component_info(nir_shader *producer, nir_shader *consumer, vc_info->is_32bit = glsl_type_is_32bit(type); vc_info->is_patch = in_var->data.patch; vc_info->is_intra_stage_only = false; + vc_info->initialised = true; } } } @@ -604,8 +605,14 @@ gather_varying_component_info(nir_shader *producer, nir_shader *consumer, unsigned var_info_idx = store_varying_info_idx[location][out_var->data.location_frac]; - if (!var_info_idx) - continue; + if (!var_info_idx) { + /* Something went wrong, the shader interfaces didn't match, so + * abandon packing. This can happen for example when the + * inputs are scalars but the outputs are struct members. + */ + *varying_comp_info_size = 0; + break; + } struct varying_component *vc_info = &(*varying_comp_info)[var_info_idx-1]; @@ -624,10 +631,23 @@ gather_varying_component_info(nir_shader *producer, nir_shader *consumer, vc_info->is_32bit = glsl_type_is_32bit(type); vc_info->is_patch = out_var->data.patch; vc_info->is_intra_stage_only = true; + vc_info->initialised = true; } } } } + + for (unsigned i = 0; i < *varying_comp_info_size; i++ ) { + struct varying_component *vc_info = &(*varying_comp_info)[i]; + if (!vc_info->initialised) { + /* Something went wrong, the shader interfaces didn't match, so + * abandon packing. This can happen for example when the outputs are + * scalars but the inputs are struct members. + */ + *varying_comp_info_size = 0; + break; + } + } } static void |