fix(util/serializer): nontermination on corrupted files
Signed-off-by: Leonardo de Moura <leonardo@microsoft.com>
This commit is contained in:
parent
d5ddb186d9
commit
f67eab000b
1 changed files with 6 additions and 4 deletions
|
@ -27,6 +27,10 @@ void serializer_core::write_int(int i) {
|
||||||
|
|
||||||
#define BIG_BUFFER 1024
|
#define BIG_BUFFER 1024
|
||||||
|
|
||||||
|
void throw_corrupted_file() {
|
||||||
|
throw exception("corrupted binary file");
|
||||||
|
}
|
||||||
|
|
||||||
void serializer_core::write_double(double d) {
|
void serializer_core::write_double(double d) {
|
||||||
std::ostringstream out;
|
std::ostringstream out;
|
||||||
// TODO(Leo): the following code may miss precision.
|
// TODO(Leo): the following code may miss precision.
|
||||||
|
@ -44,6 +48,8 @@ std::string deserializer_core::read_string() {
|
||||||
char c = m_in.get();
|
char c = m_in.get();
|
||||||
if (c == 0)
|
if (c == 0)
|
||||||
break;
|
break;
|
||||||
|
if (c == EOF)
|
||||||
|
throw_corrupted_file();
|
||||||
r += c;
|
r += c;
|
||||||
}
|
}
|
||||||
return r;
|
return r;
|
||||||
|
@ -70,8 +76,4 @@ double deserializer_core::read_double() {
|
||||||
in >> r;
|
in >> r;
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|
||||||
void throw_corrupted_file() {
|
|
||||||
throw exception("corrupted binary file");
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue