(btw, isn't it possible for you to send emails only with plain text

part?  Your emails have two parts, html and plain text, but the html one
is shown with proportional font and indents are fuzzy.  The plain text
one looks auto-generated(?) and is wrapped excessively..  Or is my
Thunderbird setting not good?)

>> +module_memory: /* TODO: test this */
>> +       for (i = 0; i < st->mods_installed; i++) {
>> +               lm = &st->load_modules[i];
>> +
>> +               for (j = MOD_TEXT; j < MOD_MEM_NUM_TYPES; j++) {
>> +                       start = lm->mem[j].base;
>> +                       end = start + lm->mem[j].size;
>> +
>> +                       if (vaddr >= end)
>> +                               continue;
>> +
>> +                       if (vaddr < start) {
>> +                               if (start < min) /* replace candidate */
>> +                                       min = start;
>>
>
> If so,  wouldn't it be overwritten in each module memory type loop? And
> later it only checked them one time(see the code at the end of this
> function).
>
> In addition, why does it need to deal with this one here? Could you please
> explain more details?

sorry, I don't understand your questions.

This function searches for the next lowest module address from the
vaddr.  Now the regions of "a module" are not one block and even not
sorted, this function needs to search all regions of all modules.  min
is a candidate, so when a lower start address than min is found, min is
replaced with the start address.