diff mbox

bitstreamout: fix for crash

Message ID 200503241922.14894.marcel.wiesweg@gmx.de
State New
Headers

Commit Message

Marcel Wiesweg March 24, 2005, 6:22 p.m. UTC 
  Hi,

the attached patch fixes an uninitialized value which made for nasty and 
reliable crashes on my machine. Strange no one came across this one before.

Marcel

Comments

Dr. Werner Fink March 24, 2005, 9:14 p.m. UTC | #1 
On Thu, Mar 24, 2005 at 07:22:14PM +0100, Marcel Wiesweg wrote:
> Hi,
> 
> the attached patch fixes an uninitialized value which made for nasty and 
> reliable crashes on my machine. Strange no one came across this one before.
> 
> Marcel

> --- bytes.h_orig        2005-03-24 18:46:37.308704736 +0100
> +++ bytes.h     2005-03-24 18:47:12.205399632 +0100
> @@ -55,5 +55,5 @@
>      }
>  public:
> -    cHandle(const uint_8 *buf, size_t len) : curr(buf), avail(len), ex(1) {};
> +    cHandle(const uint_8 *buf, size_t len) : curr(buf), avail(len), offset(0), ex(1) {};
>      inline operator uint_8  () { return check(1) ? *curr : 0; };
>      inline operator uint_16 () { return check(2) ? ntohs(*((const uint_16*)curr)) : 0; };

Oops .. thanks a lot.  This is the first report of such kind.

           Werner
diff mbox

Patch

--- bytes.h_orig        2005-03-24 18:46:37.308704736 +0100
+++ bytes.h     2005-03-24 18:47:12.205399632 +0100
@@ -55,5 +55,5 @@ 
     }
 public:
-    cHandle(const uint_8 *buf, size_t len) : curr(buf), avail(len), ex(1) {};
+    cHandle(const uint_8 *buf, size_t len) : curr(buf), avail(len), offset(0), ex(1) {};
     inline operator uint_8  () { return check(1) ? *curr : 0; };
     inline operator uint_16 () { return check(2) ? ntohs(*((const uint_16*)curr)) : 0; };