More defensive version of nsScriptablePeer::GetPlayState
Jari Korhonen <[email protected]>
| Newsgroups | gmane.comp.mozilla.mplayerplug-in |
|---|---|
| Message-ID | <[email protected]> |
Hello,
Included html file (clown_hermanni.htm) with link to Finnish
Broadcasting Co so-called "elävä arkisto" (living archive) seems to
rather reliably crash mplayerplug-in (daily build 5th Oct, 2007) in
function nsScriptablePeer::GetPlayState()
After some debugging, I found out that the it crashes because (for some
reason or other) variable mPlugin can be NULL when this function is called.
I changed function to be a little bit more defensive:
NS_IMETHODIMP nsScriptablePeer::GetPlayState(PRInt32 * aPlayState)
{
printf("JS playState issued\n");
assert (aPlayState != NULL);
//Apparently, mPlugin CAN be NULL here for some reason (external bug
?).
//Try to avoid crash if so
if (mPlugin != NULL)
mPlugin->GetPlayState(aPlayState);
else
*aPlayState = JS_STATE_UNDEFINED;
return NS_OK;
}
with this change, "elävä arkisto" seems to work ok now.
Cheers,
Jari
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Mplayerplug-in-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel
clown_hermanni.htm
(text/html, 2 KB)
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fi" lang="fi">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>yle.fi - Elävä arkisto</title>
<meta http-equiv="Pragma" content="no-cache" />
</head>
<body class="font-normal" onload="korkeus('lift1','lift2','lift3','lift4','lift5','lift6')">
</p>
<h1>Hermanni ja sirkustarkastaja</h1>
<p class="extension">1982</p>
<div class="lead">
<span class="img"><img src="kuvat/artikkeli/04175_1iso.jpg" alt="Kuva: Sirkustarkastaja (Antti Seppä) ja pelle Hermanni (Veijo Pasanen) 1982. YLE kuvanauha" width="243" height="162" /></span>
<ul class="clips">
<li class="video"><a href="javascript:;" onClick="javascript:playerpopup=window.open('http://www.yle.fi/player/player.jsp?name=El%E4v%E4+arkisto%2F04175_1','yle_player','resizable=yes,width=835,height=660');playerpopup.focus();">Sirkuspelle Hermanni</a> 01.12.1982 | kesto: 8.47<span>(Video)</span></li>
</ul>
<div class="ingres">
<p>Sirkus Hepokattiin saapuu sirkustarkastaja, jonka mukaan Hermannin mitat ovat määräysten vastaisia. Tarkastus ei olekaan mikään pelleilyn asia.</p>
</div>
<p>Sirkus-aliväli-tarkastaja Pikku Tarkkasen mukaan Hermannilla on ensiksikin määräysten vastaiset kengät. Ne ovat sirkuspellesäännöstön mukaisesti aivan liian lituskaiset.<br />
<br />
Pellen paperiasiatkaan eivät ole kunnossa. Hermannilla kun ei ole esittää opetusvalistusministeriön sirkusosaston vaatimaa pelleilylupaa, pelleopiston tai koulun päästötodistusta, ei pellediplomia eikä pellevälikirjaakaan.<br />
<br />
Kaiken kukkuraksi pellen ihannemittataulukkoon verrattuna Hermannilla on liian pitkät kädet, liian muhkuraiset polvet, kaula on liian paksu ja suu liian suuri. Niuholla tarkastajalla tuntuu olevan huomauttamista joka asiasta.<br />
<br />
Video on lyhennetty Pikku Kakkonen -ohjelmassa esitetystä jaksosta.<br />
<br />
Teksti: Elina Yli-Ojanperä</p>
</body>
</html>