mantarimay/Aegisub
1
0
Fork 0
forked from mia/Aegisub
Code Pull requests Activity

Fix use of uninitialized variable when setting margins to an empty string

Browse source 
Originally committed to SVN as r6322.
This commit is contained in:
Thomas Goyne 2012-01-20 05:14:56 +00:00
parent 4675dbb29d
commit 2bc43eb886
1 changed files with 2 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
aegisub/src/ass_dialogue.cpp
View file

@ -372,15 +372,9 @@ void AssDialogue::SetMarginString(const wxString origvalue,int which) {
} }
// Get value // Get value
long value; long value = 0;
strvalue.ToLong(&value); strvalue.ToLong(&value);
Margin[which] = mid<int>(0, value, 9999);
// Cap it
if (value < 0) value = 0;
if (value > 9999) value = 9999;
// Assign
Margin[which] = value;
} }
wxString AssDialogue::GetMarginString(int which,bool pad) const { wxString AssDialogue::GetMarginString(int which,bool pad) const {