[PATCH 06/20] libcsupport/src/mount.c: Remove unnecessary operation

Joel Sherrill joel.sherrill at oarcorp.com
Wed Nov 26 15:18:59 UTC 2014



On November 26, 2014 9:17:34 AM CST, Gedare Bloom <gedare at rtems.org> wrote:
>Again, I don't know why this is unecessary. Because str is never used?

Yes.

>On Tue, Nov 25, 2014 at 6:02 PM, Joel Sherrill
><joel.sherrill at oarcorp.com> wrote:
>> From: Josh Oguin <josh.oguin at oarcorp.com>
>>
>> This was flagged by CodeSonar.
>> ---
>>  cpukit/libcsupport/src/mount.c | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/cpukit/libcsupport/src/mount.c
>b/cpukit/libcsupport/src/mount.c
>> index d993686..0d66477 100644
>> --- a/cpukit/libcsupport/src/mount.c
>> +++ b/cpukit/libcsupport/src/mount.c
>> @@ -77,7 +77,6 @@ static rtems_filesystem_mount_table_entry_t
>*alloc_mount_table_entry(
>>
>>      memcpy( str, target, target_size );
>>      mt_entry->target = str;
>> -    str += target_size;
>>
>>      mt_entry->mounted = true;
>>      mt_entry->mt_fs_root = mt_fs_root;
>> --
>> 1.9.3
>>
>> _______________________________________________
>> devel mailing list
>> devel at rtems.org
>> http://lists.rtems.org/mailman/listinfo/devel



More information about the devel mailing list