Indeed, I had fixed that mistakes.

Patch attached.

-----Original Message-----
From: Vincent Torri [mailto:vincent.to...@gmail.com] 
Sent: 17 wrzeĊ›nia 2012 19:30
To: Enlightenment developer list
Subject: Re: [E-devel] [Patch] [Ecore] Patch with NULL checks to some
functions in ecore module

On Mon, Sep 17, 2012 at 7:29 PM, Vincent Torri <vincent.to...@gmail.com>
wrote:
> On Mon, Sep 17, 2012 at 5:19 PM, Patryk Kaczmarek <patry...@samsung.com>
wrote:
>> Hi Everyone,
>>
>>
>>
>> I had prepared patch for some Ecore functions which, in my opinion, 
>> critically need NULL checks. I had tried to keep each file convention.
>>
>>
>>
>> Then please review and consider the attachment and give some feedback.
>
> in the ecore_file_download.c part, 'dir' should be defined after the 
> test on 'url', and not during its declaration.

same for 'fp' in ecore_evas_util.c part

Vincent

----------------------------------------------------------------------------
--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and threat
landscape has changed and how IT managers can respond. Discussions will
include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Attachment: 0001-Ecore-null-checks.patch
Description: Binary data

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to