Skip to content

Commit

Permalink
fixes #86
Browse files Browse the repository at this point in the history
  • Loading branch information
@marekkokot
marekkokot committed Jul 17, 2018
1 parent 981e2fd commit f56a92a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion kmc_api/kmc_file.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ bool CKMCFile::OpenForRA(const std::string &file_name)

sufix_file_buf = new uchar[size];
result = fread(sufix_file_buf, 1, size, file_suf);
if (result == 0)
if (result != size)
return false;

fclose(file_suf);
Expand Down

2 comments on commit f56a92a

@notestaff
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marekkokot great, thanks. can you tag a new release with these bugfixes, so that I can update the bioconda recipe to install the fixed version? if not ready for a full numbered release it can be a pre-release e.g. 3.1.0-rc.1 ; just needs a tagged name.

@marekkokot
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's done. BTW we will upgrade to GPL 3, but again I don't know when :( It should resolve some licensing issues.

Please sign in to comment.