[Gnash-commit] [SCM] Gnash branch, master, updated. release_0_8_9_final-1625-gae47be9

[Bastiaan Jacques]

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  ae47be9a85eed55aaa99bba1c12959e249f06944 (commit)
      from  23c47c8f5904f8543932b89b8165d4f16b810cac (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=ae47be9a85eed55aaa99bba1c12959e249f06944


commit ae47be9a85eed55aaa99bba1c12959e249f06944
Author: Bastiaan Jacques <address@hidden>
Date:   Sat Jun 1 11:58:50 2013 +0200

    Fix potential out-of-bounds vector access.
    
    If |index == _amfobjs.size()| then |_amfobjs[index]| is out of bounds. 
Found using cppcheck.

diff --git a/cygnal/libnet/rtmp_msg.cpp b/cygnal/libnet/rtmp_msg.cpp
index 47371c0..a606137 100644
--- a/cygnal/libnet/rtmp_msg.cpp
+++ b/cygnal/libnet/rtmp_msg.cpp
@@ -169,7 +169,7 @@ boost::shared_ptr<cygnal::Element>
 RTMPMsg::operator[](size_t index)
 {
 //    GNASH_REPORT_FUNCTION;
-    if (index <= _amfobjs.size()) {
+    if (index < _amfobjs.size()) {
        return _amfobjs[index];
     }
     

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

Summary of changes:
 cygnal/libnet/rtmp_msg.cpp |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Gnash