Comment 18 for bug 545336

Revision history for this message
Michael Vogt (mvo) wrote :

Thanks for the bugreport and thanks to Jean-Baptiste Lallement for analyzing the problem.

The bug here is the following bit in debrecords.cc:

                  Tags(&File,Cache.Head().MaxVerFileSize + 200)

MaxVerFileSize is the maximum size for a package record found in the Packages file. This is not sufficient
in the days of Translations-$lang and we need to update the code so that it takes the translation files into
consideration for the maximum size too. This bug has not shown yet because the code already uses the
biggest record. In this case that was probably t-l-e already and even with the additional meta-data from
the packages file (like dependencies etc) it is not enough.

I see two possible fixes:

Simple:
- Tags(&File,Cache.Head().MaxVerFileSize + 200)
+ Tags(&File,Cache.Head().MaxVerFileSize * 2)

Better:
- Tags(&File,Cache.Head().MaxVerFileSize + 200)
+ Tags(&File, max(Cache.Head().MaxVerFileSize,Cache.Head().MaxDescFileSize) + 200)