[Glitch] Improve safety of Blurhash component

Port 66f9bd08da to glitch-soc

Signed-off-by: Thibaut Girka <thib@sitedethib.com>
th-downstream
Sasha Sorokin 4 years ago committed by Thibaut Girka
parent f2d0bd912c
commit c952cc3115

@ -6,7 +6,7 @@ import PropTypes from 'prop-types';
/** /**
* @typedef BlurhashPropsBase * @typedef BlurhashPropsBase
* @property {string} hash Hash to render * @property {string?} hash Hash to render
* @property {number} width * @property {number} width
* Width of the blurred region in pixels. Defaults to 32 * Width of the blurred region in pixels. Defaults to 32
* @property {number} [height] * @property {number} [height]
@ -37,13 +37,17 @@ function Blurhash({
const { current: canvas } = canvasRef; const { current: canvas } = canvasRef;
canvas.width = canvas.width; // resets canvas canvas.width = canvas.width; // resets canvas
if (dummy) return; if (dummy || !hash) return;
const pixels = decode(hash, width, height); try {
const ctx = canvas.getContext('2d'); const pixels = decode(hash, width, height);
const imageData = new ImageData(pixels, width, height); const ctx = canvas.getContext('2d');
const imageData = new ImageData(pixels, width, height);
ctx.putImageData(imageData, 0, 0); ctx.putImageData(imageData, 0, 0);
} catch (err) {
console.error('Blurhash decoding failure', { err, hash });
}
}, [dummy, hash, width, height]); }, [dummy, hash, width, height]);
return ( return (

Loading…
Cancel
Save