Hey guys,
attached patch fixes calculation of the Content-Length header,
which didn't account for the header size of "mdat" atom(s)
in case of seeked mp4 files.

Best regards,
Piotr Sikora < piotr.sikora@frickle.com >

--- src/http/modules/ngx_http_mp4_module.c.orig Sun Nov 13 04:56:07 2011
+++ src/http/modules/ngx_http_mp4_module.c Sun Nov 13 04:56:26 2011
@@ -1066,7 +1066,6 @@

atom_data_size = mp4->mdat_data.buf->file_last - start_offset;
mp4->mdat_data.buf->file_pos = start_offset;
- mp4->content_length += atom_data_size;

ngx_log_debug2(NGX_LOG_DEBUG_HTTP, mp4->file.log, 0,
"mdat new offset @%O:%O", start_offset, atom_data_size);
@@ -1082,6 +1081,8 @@
atom_size = sizeof(ngx_mp4_atom_header_t) + atom_data_size;
atom_header_size = sizeof(ngx_mp4_atom_header_t);
}
+
+ mp4->content_length += atom_header_size + atom_data_size;

ngx_mp4_set_32value(atom_header, atom_size);
ngx_mp4_set_atom_name(atom_header, 'm', 'd', 'a', 't');
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel