Gnash - The GNU Flash player - Bugs: bug #43867, int overflow

Group

Main
- Main
- View members
- Search
Homepage
Download
Support
- Submit new
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Mailing lists
Source code
Bugs
- Submit new
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Tasks
- ~~Submit new~~
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Patches
- ~~Submit new~~
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
News
- Browse
- Atom feed
- ~~Submit~~
- ~~Manage~~

bug #43867: int overflow

Submitter:	Joshua Rogers <megamansec3>
Submitted:	Mon 22 Dec 2014 11:46:38 AM UTC


Category:	core	Severity:	3 - Normal
Release:	None	Status:	None
Privacy:	Public	Assigned to:	None
Open/Closed:	Open

* Mandatory Fields

Add a New Comment Rich Markup

Mon 22 Dec 2014 10:41:18 PM UTC, comment #1: It seems that if such an overflow occurs, and some bogus value is passed to construct_object, eventually Gnash will die with an exception because of too many calls to SafeStack::pop(). As far as I can tell there is no validation of nargs even if its value isn't the result of an overflow.	Bastiaan Jacques <bjacques>
Mon 22 Dec 2014 11:46:38 AM UTC, original submission: Hi, In ASHandlers.cpp: 2306 unsigned nargs = toNumber(env.pop(), getVM(env)); may cause an int overflow dur to the conversation from 64bits to 32bits And then it is used: 2325 as_object* newobj = construct_object(constructor, env, nargs); which will cause problems. Thanks,	Joshua Rogers <megamansec3>

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:


Comment:

No files currently attached

Depends on the following items: None found

Items that depend on this one: None found

Carbon-Copy List

-email is unavailable- added by bjacques (Posted a comment)

-email is unavailable- added by megamansec3 (Submitted the item)

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

Please enter the title of George Orwell's famous dystopian book (it's a date):

Follow 2 latest changes.

Date	Changed by	Updated Field	Previous Value	=>	Replaced by
2014-12-22	bjacques	Category	None		core
		Privacy	Private		Public

Gnash - The GNU Flash player - Bugs: bug #43867, int overflow

bug #43867: int overflow

<a href="#postcomment">Post a Comment</a>

<a href="#discussion">Discussion</a>

<a href="#attached">Attached Files</a>

<a href="#dependencies">Dependencies</a>

<a href="#cc">Mail Notification Carbon-Copy List</a>

<a href="#votes">Votes</a>

<a href="#history">History</a>