Re: [Mesa-dev] [PATCH 2/2] mesa: Handle negative length in glPushDebugGroup().

2016-03-24 Thread Eduardo Lima Mitev
Both patches are: Reviewed-by: Eduardo Lima Mitev Together with the other patch you sent earlier, I get these results for dEQP-GLES31.functional.debug.* : Test run totals: Passed:1004/1010 (99.4%) Failed:6/1010 (0.6%) Not supported: 0/1010 (0.0%)

[Mesa-dev] [PATCH 2/2] mesa: Handle negative length in glPushDebugGroup().

2016-03-24 Thread Kenneth Graunke
The KHR_debug spec doesn't actually say we should handle this, but that is most likely an oversight - it says to check against strlen and generate errors if length is negative. It appears they just forgot to explicitly spell out that we should then proceed to actually handle it. Fixes crashes