#3739 closed defect (fixed)
ALL: config manager crashes when reading too long lines
Reported by: | eriktorbjorn | Owned by: | fingolfin |
---|---|---|---|
Priority: | high | Component: | --Other-- |
Version: | Keywords: | ||
Cc: | Game: |
Description
Latest SVN snapshot
It is possible (albeit unlikely) for the config manager to create a config file that cannot be read because the lines are too long. I'm speculating that this is what happened in the following forum thread:
http://forums.scummvm.org/viewtopic.php?t=5679
Where it looks like someone used an extremely long path name for Zak McKracken.
The simplest "solution" is, of course, to simply increase the maximum line length, to make it even less likely to happen. But that doesn't seem pretty.
Ticket imported from: #1971499. Ticket imported from: bugs/3739.
Change History (7)
comment:1 by , 16 years ago
comment:2 by , 16 years ago
Summary: | The config manager can create files with too long lines → ALL: config manager can create files with too long lines |
---|
comment:3 by , 16 years ago
Priority: | normal → high |
---|
comment:5 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Summary: | ALL: config manager can create files with too long lines → ALL: config manager crashes when reading too long lines |
comment:6 by , 16 years ago
Config manager code has been fixed to handle reading of arbitrarily long lines.
comment:7 by , 6 years ago
Component: | → --Other-- |
---|
Well, either we should support "arbitrarily" long lines, or we should prevent this from happening in the first place...