[Gnash-commit] [SCM] Gnash branch, master, updated. 2336bcda55a47842b85569c11e94a83348fdcac7

[Sandro Santilli]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Gnash".

The branch, master has been updated
       via  2336bcda55a47842b85569c11e94a83348fdcac7 (commit)
      from  dee7acf1d89b97592479159ccb9a3473c61c1b0f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.savannah.gnu.org/cgit//commit/?id=2336bcda55a47842b85569c11e94a83348fdcac7


commit 2336bcda55a47842b85569c11e94a83348fdcac7
Author: Sandro Santilli <address@hidden>
Date:   Mon Oct 4 17:45:17 2010 +0200

    If cookie is null, ncookie (string) constructor would segfault... so check 
before trying

diff --git a/plugin/npapi/plugin.cpp b/plugin/npapi/plugin.cpp
index b73ee3b..a3dac9b 100644
--- a/plugin/npapi/plugin.cpp
+++ b/plugin/npapi/plugin.cpp
@@ -974,8 +974,8 @@ nsPluginInstance::setupCookies(const std::string& pageurl)
     uint32_t length = 0;
     NPN_GetValueForURL(_instance, NPNURLVCookie, url.c_str(),
                        &cookie, &length);
-    std::string ncookie (cookie, length);
     if (cookie) {
+        std::string ncookie (cookie, length);
         gnash::log_debug("The Cookie for %s is %s", url, ncookie);
         std::ofstream cookiefile;
         std::stringstream ss;

-----------------------------------------------------------------------

Summary of changes:
 plugin/npapi/plugin.cpp |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Gnash