| Summary: | Build issue: kftpgrabber fails in rpmlint | ||
|---|---|---|---|
| Product: | TDE | Reporter: | Francois Andriot <albator78> |
| Component: | non-core programs | Assignee: | Michele Calgaro <michele.calgaro> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | albator78, bugwatch, michele.calgaro, slavek.banko |
| Priority: | P5 | ||
| Version: | R14.0.x [Trinity] | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| Compiler Version: | TDE Version String: | ||
| Application Version: | Application Name: | ||
| Bug Depends on: | |||
| Bug Blocks: | 3010 | ||
| Attachments: | kftpgrabber: fix rpmlint check test | ||
|
Description
Francois Andriot
2015-04-03 15:33:21 CDT
Created attachment 2474 [details]
kftpgrabber: fix rpmlint check test
The attached patches fixes both issues.
(patches taken from opensuse-kde3)
I fear that this patch is incorrect. I tried simple code:
#include <stdio.h>
#include <stdlib.h>
main()
{
char* a10 = (char*) malloc(10);
char s10[10];
printf("size allocated: %d\n", sizeof a10);
printf("size static: %d\n", sizeof s10);
}
Compiled using g++ (on 32bit system) returns:
size allocated: 4
size static: 10
Therefore I fear that the adjustments contained in patch are incorrect:
memset(o_message, 0, sizeof o_message);
memset(o_output, 0, sizeof o_output);
Variables o_message and o_output are allocated by malloc, I excpect, that sizeof will return size of pointer - 4 (on 32bit system).
Proposed fix here: https://mirror.git.trinitydesktop.org/gitea/TDE/kftpgrabber/pulls/1 |