aaru-dps/Aaru
1
0
Fork 0
mirror of https://github.com/aaru-dps/Aaru.git synced 2025-12-16 19:24:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Correct media scan statistics check of arguments in range.

Browse Source
This commit is contained in:
Natalia Portillo
2018-11-22 07:14:38 +00:00
parent 562e86fbcb
commit b2233c794f
1 changed files with 18 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
DiscImageChef.Core/Statistics.cs
View File

@@ -83,7 +83,10 @@ namespace DiscImageChef.Core
version = DetectOS.GetVersion() version = DetectOS.GetVersion()
} }
}, },
Versions = new List<NameValueStats> {new NameValueStats {name = Version.GetVersion(), Value = 1}} Versions = new List<NameValueStats>
{
new NameValueStats {name = Version.GetVersion(), Value = 1}
}
}; };
XmlSerializer xs = new XmlSerializer(AllStats.GetType()); XmlSerializer xs = new XmlSerializer(AllStats.GetType());
StreamReader sr = new StreamReader(Path.Combine(Settings.Settings.StatsPath, "Statistics.xml")); StreamReader sr = new StreamReader(Path.Combine(Settings.Settings.StatsPath, "Statistics.xml"));
@@ -105,7 +108,10 @@ namespace DiscImageChef.Core
version = DetectOS.GetVersion() version = DetectOS.GetVersion()
} }
}, },
Versions = new List<NameValueStats> {new NameValueStats {name = Version.GetVersion(), Value = 1}} Versions = new List<NameValueStats>
{
new NameValueStats {name = Version.GetVersion(), Value = 1}
}
}; };
} }
else else
@@ -669,8 +675,7 @@ namespace DiscImageChef.Core
foreach(ChecksumStats st in checksums.Select(kvp => new ChecksumStats foreach(ChecksumStats st in checksums.Select(kvp => new ChecksumStats
{ {
algorithm = kvp.Key, algorithm = kvp.Key, Value = kvp.Value
Value = kvp.Value
})) }))
{ {
CurrentStats.Benchmark.Checksum.Add(st); CurrentStats.Benchmark.Checksum.Add(st);
@@ -748,7 +753,15 @@ namespace DiscImageChef.Core
long lessThan500Ms, long moreThan500Ms, long total, long error, long lessThan500Ms, long moreThan500Ms, long total, long error,
long correct) long correct)
{ {
if(lessThan500Ms <= 0) throw new ArgumentOutOfRangeException(nameof(lessThan500Ms)); if(lessThan3Ms < 0) throw new ArgumentOutOfRangeException(nameof(lessThan3Ms));
if(lessThan10Ms < 0) throw new ArgumentOutOfRangeException(nameof(lessThan10Ms));
if(lessThan50Ms < 0) throw new ArgumentOutOfRangeException(nameof(lessThan50Ms));
if(lessThan150Ms < 0) throw new ArgumentOutOfRangeException(nameof(lessThan150Ms));
if(lessThan500Ms < 0) throw new ArgumentOutOfRangeException(nameof(lessThan500Ms));
if(moreThan500Ms < 0) throw new ArgumentOutOfRangeException(nameof(moreThan500Ms));
if(total < 0) throw new ArgumentOutOfRangeException(nameof(total));
if(error < 0) throw new ArgumentOutOfRangeException(nameof(error));
if(correct < 0) throw new ArgumentOutOfRangeException(nameof(correct));
if(Settings.Settings.Current.Stats == null || !Settings.Settings.Current.Stats.MediaScanStats) return; if(Settings.Settings.Current.Stats == null || !Settings.Settings.Current.Stats.MediaScanStats) return;