linuxdcpp-team team mailing list archive
-
linuxdcpp-team team
-
Mailing list archive
-
Message #08934
[Bug 1815377] Re: SFV checker file name bug
You're always welcome to create a patch. I think these days that'd be
the fastest solution for yourself especially if you still keep using
DC++ code as upstream.
--
You received this bug notification because you are a member of
Dcplusplus-team, which is subscribed to DC++.
https://bugs.launchpad.net/bugs/1815377
Title:
SFV checker file name bug
Status in DC++:
Confirmed
Bug description:
SFV checker code contains a bug: the file name inside SFV file is
searched by substring and no check for end of line is performed.
Example:
The folder with the following files:
the.fp.2011.limited.1080p.bluray.x264-psychd.sub
the.fp.2011.limited.1080p.bluray.x264-psychd.subs.sfv
SFV file contains:
the.fp.2011.limited.1080p.bluray.x264-psychd.subs.rar 18239c0b
Expected result:
No CRC32 record for "the.fp.2011.limited.1080p.bluray.x264-psychd.sub" is found inside SFV file.
Actual result:
dcpp/SFVReader.cpp SFVReader::tryFile opens SFV file, finds "the.fp.2011.limited.1080p.bluray.x264-psychd.sub" substring and tries to convert hex string of "s.rar 18" (instead of "18239c0b") to hex value with sscanf.
the.fp.2011.limited.1080p.bluray.x264-psychd.sub is not added to share because of incorrect CRC32.
To manage notifications about this bug go to:
https://bugs.launchpad.net/dcplusplus/+bug/1815377/+subscriptions
References