Merged,
Maxim.
On 01/25/17 05:33, Bill Fischofer wrote:
> On Mon, Jan 23, 2017 at 2:25 AM, Christophe Milard
> <christophe.mil...@linaro.org> wrote:
>> Hence fixing CID 174663
>> (Fixes https://bugs.linaro.org/show_bug.cgi?id=2827)
>>
>> Signed-off-by: Christophe Milard <christophe.mil...@linaro.org>
>
> Reviewed-by: Bill Fischofer <bill.fischo...@linaro.org>
>
>> ---
>>
>> since V1:
>> -print strerror(errno) and sets __odp_errno (Maxim)
>> -ref to bugzilla bug ID (Mike, Bill)
>>
>> platform/linux-generic/_ishm.c | 9 ++++++++-
>> 1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/platform/linux-generic/_ishm.c b/platform/linux-generic/_ishm.c
>> index f889834..5ca1067 100644
>> --- a/platform/linux-generic/_ishm.c
>> +++ b/platform/linux-generic/_ishm.c
>> @@ -818,7 +818,14 @@ int _odp_ishm_reserve(const char *name, uint64_t size,
>> int fd,
>>
>> /* If a file descriptor is provided, get the real size and map: */
>> if (fd >= 0) {
>> - fstat(fd, &statbuf);
>> + if (fstat(fd, &statbuf) < 0) {
>> + close(fd);
>> + odp_spinlock_unlock(&ishm_tbl->lock);
>> + ODP_ERR("_ishm_reserve failed (fstat failed: %s).\n",
>> + strerror(errno));
>> + __odp_errno = errno;
>> + return -1;
>> + }
>> len = statbuf.st_size;
>> /* note that the huge page flag is meningless here as huge
>> * page is determined by the provided file descriptor: */
>> --
>> 2.7.4
>>