On Fri, May 27, 2011 at 12:14 PM, devzero2000 <pinto.e...@gmail.com> wrote:

> On Fri, May 27, 2011 at 9:40 AM, Qing He <qing...@intel.com> wrote:
>
>> In fpLookupSubdir, data returned by hash should be of type
>> "struct rpmffi_s **" instead of "struct rpmffi_s *" to avoid
>> segfault.
>>
>> Thanks,
>> Qing
>>
>> Fixed already with commit  db407ba1f54292552bafc13a8989238e262e7470
> in rpm5.3, i presume.
>
> commit db407ba1f54292552bafc13a8989238e262e7470
> Author: jbj <jbj>
> Date:   Sat Jan 30 20:19:53 2010 +0000
>
>     - haul out some rpm.org trash.
>
> What version are you using ? I will backport where necessary. I will take a
> look better.
>
Sorry, i have looked bad . I was without glass. I will merge shortly. Thanks

>
> Thanks
>
> Best Regards
>
>> ---
>> diff --git a/rpmdb/fprint.c b/rpmdb/fprint.c
>> index 0e76148..82b8f45 100644
>> --- a/rpmdb/fprint.c
>> +++ b/rpmdb/fprint.c
>> @@ -333,7 +333,7 @@ restart:
>>     *te = '\0';
>>
>>     while (te < se) {
>> -       struct rpmffi_s * recs;
>> +       struct rpmffi_s ** recs;
>>        int numRecs;
>>        int i;
>>
>> @@ -346,8 +346,8 @@ restart:
>>            const char * link;
>>            int fx;
>>
>> -           fx = recs[i].fileno;
>> -           fi =  recs[i].p->fi;
>> +           fx = recs[i]->fileno;
>> +           fi =  recs[i]->p->fi;
>>            flink = fi->flinks[fx];
>>            if (!(flink && *flink != '\0'))
>>
>                continue;
>> ______________________________________________________________________
>> RPM Package Manager                                    http://rpm5.org
>> Developer Communication List                        rpm-devel@rpm5.org
>>
>
>

Reply via email to