I know that this bug report came before 0013247, but that bug report has a solution whereas this doesn't.

Sorry, not a duplicate. Repoening.

File proj_doc_update.php line 105

Change:-

case DATABASE:
$c_content = db_prepare_binary_string( fread ( fopen( $v_tmp_name, 'rb' ), $v_size ) );
break;

to:-

case DATABASE:
$c_content = fread ( fopen( $v_tmp_name, 'rb' ), $v_size );
break;

I changed line 105 and line 54 (0013247). This works for me.

David, I'm afraid that your proposed fix would only work for MySQL - other DB engines rely on BLOB fields to be escaped/encoded properly (look at db_prepare_binary_string).

I think I figured it out. See https://github.com/dregad/mantisbt/commits/file-attach-fixes - testing and feedback welcome :-)

How to download a tarball with the "file-attach-fixes"? Please provide a link or a git-command.

You can download it from the link is in my previous post 0012955:0032538
Also https://github.com/dregad/mantisbt/tarball/file-attach-fixes

I tested the software from https://github.com/dregad/mantisbt/tarball/file-attach-fixes, problem is solved, no damaged files anymore.

Thanks for your feedback, I'll include that in the next release then.

Just upgraded to 1.2.15 and this is still broken.
Surely fixing it for MySQL is better than leaving it broken. The non-MySQL users will be in just the same position as they are now.

David

This is targeted at 1.2.x which means it would be included in 1.2.16 -- assuming I get around to merging the changes ;-)

Until then, feel free to apply the patch from my github branch on top of your 1.2.15 instance, and please report here if you experience any issues.