Re: msevior - r25767 - abiword/branches/ABI-2-6-0-STABLE/src/wp/ap/unix

From: Martin Sevior <msevior_at_gmail.com>
Date: Sat Feb 28 2009 - 00:03:09 CET

Good point. Will fix.

On Sat, Feb 28, 2009 at 9:43 AM, Hubert Figuiere <hfiguiere@teaser.fr> wrote:
>
> On 02/27/2009 05:36 PM, cvs@abisource.com wrote:
>>
>>  extern "C" GdkPixbuf *
>>  abi_widget_render_page_to_image(AbiWidget *abi, int iPage)
>>  {
>> +       //
>> +       // AbiWord counts from 0 but we let the caller count from 1.
>> +       //
>> +       iPage--;
>
>
>
> And you are not checking the input?
>
> Given that it is an API, it would be safer to check the input against values
> <= 0
>
> Hub
>
Received on Sat Feb 28 00:03:14 2009

This archive was generated by hypermail 2.1.8 : Sat Feb 28 2009 - 00:03:14 CET