summaryrefslogtreecommitdiffstats
path: root/libass
diff options
context:
space:
mode:
authorStefano Pigozzi <stefano.pigozzi@gmail.com>2014-04-29 21:58:49 +0200
committerGrigori Goronzy <greg@chown.ath.cx>2015-07-10 10:42:40 +0200
commitf72f47fd5e2ab3832bdc3024b3a8f85d570492f6 (patch)
treec6753d3ca1a095f1e43f53a8acd21ea34e1dd7f3 /libass
parent8bebabe4a62a6d54cdfecace59c382efa80a8e47 (diff)
downloadlibass-f72f47fd5e2ab3832bdc3024b3a8f85d570492f6.tar.bz2
libass-f72f47fd5e2ab3832bdc3024b3a8f85d570492f6.tar.xz
fontselect: fix NULL dereference
Apparently on some broken fonts, FreeType can return a NULL family name. Avoid to crash in these cases.
Diffstat (limited to 'libass')
-rw-r--r--libass/ass_fontselect.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libass/ass_fontselect.c b/libass/ass_fontselect.c
index 398d8f9..bc1dec5 100644
--- a/libass/ass_fontselect.c
+++ b/libass/ass_fontselect.c
@@ -640,7 +640,7 @@ get_font_info(FT_Library lib, FT_Face face, ASS_FontProviderMetaData *info)
iconv_close(utf16to8);
// check if we got a valid family - if not use whatever FreeType gives us
- if (num_family == 0) {
+ if (num_family == 0 && face->family_name) {
families[0] = strdup(face->family_name);
num_family++;
}