divide by zero on incorrectly specified oembed thumbnail size

This commit is contained in:
friendica 2012-12-16 17:58:12 -08:00
parent 6b36863dd8
commit 3f63a69d7a
3 changed files with 10 additions and 9 deletions

View file

@ -12,7 +12,7 @@ require_once('library/Mobile_Detect/Mobile_Detect.php');
require_once('include/features.php'); require_once('include/features.php');
define ( 'FRIENDICA_PLATFORM', 'Friendica'); define ( 'FRIENDICA_PLATFORM', 'Friendica');
define ( 'FRIENDICA_VERSION', '3.1.1556' ); define ( 'FRIENDICA_VERSION', '3.1.1559' );
define ( 'DFRN_PROTOCOL_VERSION', '2.23' ); define ( 'DFRN_PROTOCOL_VERSION', '2.23' );
define ( 'DB_UPDATE_VERSION', 1157 ); define ( 'DB_UPDATE_VERSION', 1157 );

View file

@ -74,9 +74,10 @@ function oembed_format_object($j){
switch ($j->type) { switch ($j->type) {
case "video": { case "video": {
if (isset($j->thumbnail_url)) { if (isset($j->thumbnail_url)) {
$tw = (isset($j->thumbnail_width)) ? $j->thumbnail_width:200; $tw = (isset($j->thumbnail_width) && intval($j->thumbnail_width)) ? $j->thumbnail_width:200;
$th = (isset($j->thumbnail_height)) ? $j->thumbnail_height:180; $th = (isset($j->thumbnail_height) && intval($j->thumbnail_height)) ? $j->thumbnail_height:180;
$tr = $tw/$th; // make sure we don't attempt divide by zero, fallback is a 1:1 ratio
$tr = (($th) ? $tw/$th : 1);
$th=120; $tw = $th*$tr; $th=120; $tw = $th*$tr;
$tpl=get_markup_template('oembed_video.tpl'); $tpl=get_markup_template('oembed_video.tpl');

View file

@ -6,9 +6,9 @@
#, fuzzy #, fuzzy
msgid "" msgid ""
msgstr "" msgstr ""
"Project-Id-Version: 3.1.1556\n" "Project-Id-Version: 3.1.1559\n"
"Report-Msgid-Bugs-To: \n" "Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2012-12-13 10:00-0800\n" "POT-Creation-Date: 2012-12-16 10:00-0800\n"
"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n" "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
"Language-Team: LANGUAGE <LL@li.org>\n" "Language-Team: LANGUAGE <LL@li.org>\n"
@ -7836,12 +7836,12 @@ msgid "Set twitter search term"
msgstr "" msgstr ""
#: ../../view/theme/diabook/theme.php:629 #: ../../view/theme/diabook/theme.php:629
#: ../../view/theme/diabook/config.php:146 ../../include/acl_selectors.php:288 #: ../../view/theme/diabook/config.php:146 ../../include/acl_selectors.php:313
msgid "don't show" msgid "don't show"
msgstr "" msgstr ""
#: ../../view/theme/diabook/theme.php:629 #: ../../view/theme/diabook/theme.php:629
#: ../../view/theme/diabook/config.php:146 ../../include/acl_selectors.php:287 #: ../../view/theme/diabook/config.php:146 ../../include/acl_selectors.php:312
msgid "show" msgid "show"
msgstr "" msgstr ""
@ -9031,7 +9031,7 @@ msgstr ""
msgid "[no subject]" msgid "[no subject]"
msgstr "" msgstr ""
#: ../../include/acl_selectors.php:286 #: ../../include/acl_selectors.php:311
msgid "Visible to everybody" msgid "Visible to everybody"
msgstr "" msgstr ""