* [PATCH 0/2] Bug fixes for makedevs.c
@ 2019-10-28 17:14 Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 1/2] fix format strings in makedevs.c in print statements Frazer Leslie Clews
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Frazer Leslie Clews @ 2019-10-28 17:14 UTC (permalink / raw)
To: openembedded-core; +Cc: Frazer Leslie Clews
fixes a few bugs in some print statements and an overflow bug with usr_buf
Frazer Leslie Clews (2):
fix format strings in makedevs.c in print statements
fix invalidScanfFormatWidth to prevent overflowing usr_buf
meta/recipes-devtools/makedevs/makedevs/makedevs.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] fix format strings in makedevs.c in print statements
2019-10-28 17:14 [PATCH 0/2] Bug fixes for makedevs.c Frazer Leslie Clews
@ 2019-10-28 17:14 ` Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 2/2] fix invalidScanfFormatWidth to prevent overflowing usr_buf Frazer Leslie Clews
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Frazer Leslie Clews @ 2019-10-28 17:14 UTC (permalink / raw)
To: openembedded-core; +Cc: Frazer Leslie Clews
Signed-off-by: Frazer Leslie Clews <frazer.clews@codethink.co.uk>
---
meta/recipes-devtools/makedevs/makedevs/makedevs.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/meta/recipes-devtools/makedevs/makedevs/makedevs.c b/meta/recipes-devtools/makedevs/makedevs/makedevs.c
index cba7681414..01e564afee 100644
--- a/meta/recipes-devtools/makedevs/makedevs/makedevs.c
+++ b/meta/recipes-devtools/makedevs/makedevs/makedevs.c
@@ -230,7 +230,7 @@ static void add_new_directory(char *name, char *path,
unsigned long uid, unsigned long gid, unsigned long mode)
{
if (trace)
- fprintf(stderr, "Directory: %s %s UID: %ld GID %ld MODE: %04lo", path, name, uid, gid, mode);
+ fprintf(stderr, "Directory: %s %s UID: %lu GID %lu MODE: %04lo", path, name, uid, gid, mode);
if (mkdir(path, mode) < 0) {
if (EEXIST == errno) {
@@ -251,7 +251,7 @@ static void add_new_device(char *name, char *path, unsigned long uid,
struct stat sb;
if (trace) {
- fprintf(stderr, "Device: %s %s UID: %ld GID: %ld MODE: %04lo MAJOR: %d MINOR: %d",
+ fprintf(stderr, "Device: %s %s UID: %lu GID: %lu MODE: %04lo MAJOR: %d MINOR: %d",
path, name, uid, gid, mode, (short)(rdev >> 8), (short)(rdev & 0xff));
}
@@ -292,7 +292,7 @@ static void add_new_file(char *name, char *path, unsigned long uid,
unsigned long gid, unsigned long mode)
{
if (trace) {
- fprintf(stderr, "File: %s %s UID: %ld GID: %ld MODE: %04lo\n",
+ fprintf(stderr, "File: %s %s UID: %lu GID: %lu MODE: %04lo\n",
path, name, gid, uid, mode);
}
@@ -311,7 +311,7 @@ static void add_new_fifo(char *name, char *path, unsigned long uid,
unsigned long gid, unsigned long mode)
{
if (trace) {
- printf("Fifo: %s %s UID: %ld GID: %ld MODE: %04lo\n",
+ printf("Fifo: %s %s UID: %lu GID: %lu MODE: %04lo\n",
path, name, gid, uid, mode);
}
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] fix invalidScanfFormatWidth to prevent overflowing usr_buf
2019-10-28 17:14 [PATCH 0/2] Bug fixes for makedevs.c Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 1/2] fix format strings in makedevs.c in print statements Frazer Leslie Clews
@ 2019-10-28 17:14 ` Frazer Leslie Clews
2019-10-28 17:32 ` ✗ patchtest: failure for Bug fixes for makedevs.c Patchwork
2019-10-28 19:11 ` [PATCH 0/2] " Andre McCurdy
3 siblings, 0 replies; 6+ messages in thread
From: Frazer Leslie Clews @ 2019-10-28 17:14 UTC (permalink / raw)
To: openembedded-core; +Cc: Frazer Leslie Clews
Signed-off-by: Frazer Leslie Clews <frazer.clews@codethink.co.uk>
---
meta/recipes-devtools/makedevs/makedevs/makedevs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/makedevs/makedevs/makedevs.c b/meta/recipes-devtools/makedevs/makedevs/makedevs.c
index 01e564afee..32b9872932 100644
--- a/meta/recipes-devtools/makedevs/makedevs/makedevs.c
+++ b/meta/recipes-devtools/makedevs/makedevs/makedevs.c
@@ -360,7 +360,7 @@ static int interpret_table_entry(char *line)
unsigned long mode = 0755, uid = 0, gid = 0, major = 0, minor = 0;
unsigned long start = 0, increment = 1, count = 0;
- if (0 > sscanf(line, "%4095s %c %lo %40s %40s %lu %lu %lu %lu %lu", path,
+ if (0 > sscanf(line, "%4095s %c %lo %39s %39s %lu %lu %lu %lu %lu", path,
&type, &mode, usr_buf, grp_buf, &major, &minor, &start,
&increment, &count))
{
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* ✗ patchtest: failure for Bug fixes for makedevs.c
2019-10-28 17:14 [PATCH 0/2] Bug fixes for makedevs.c Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 1/2] fix format strings in makedevs.c in print statements Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 2/2] fix invalidScanfFormatWidth to prevent overflowing usr_buf Frazer Leslie Clews
@ 2019-10-28 17:32 ` Patchwork
2019-10-28 19:11 ` [PATCH 0/2] " Andre McCurdy
3 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2019-10-28 17:32 UTC (permalink / raw)
To: Frazer Leslie Clews; +Cc: openembedded-core
== Series Details ==
Series: Bug fixes for makedevs.c
Revision: 1
URL : https://patchwork.openembedded.org/series/20722/
State : failure
== Summary ==
Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:
* Patch [1/2] fix format strings in makedevs.c in print statements
Issue Shortlog does not follow expected format [test_shortlog_format]
Suggested fix Commit shortlog (first line of commit message) should follow the format "<target>: <summary>"
If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).
---
Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Bug fixes for makedevs.c
2019-10-28 17:14 [PATCH 0/2] Bug fixes for makedevs.c Frazer Leslie Clews
` (2 preceding siblings ...)
2019-10-28 17:32 ` ✗ patchtest: failure for Bug fixes for makedevs.c Patchwork
@ 2019-10-28 19:11 ` Andre McCurdy
2019-10-28 21:22 ` Ross Burton
3 siblings, 1 reply; 6+ messages in thread
From: Andre McCurdy @ 2019-10-28 19:11 UTC (permalink / raw)
To: Frazer Leslie Clews; +Cc: OE Core mailing list
On Mon, Oct 28, 2019 at 10:15 AM Frazer Leslie Clews
<frazer.clews@codethink.co.uk> wrote:
>
> fixes a few bugs in some print statements and an overflow bug with usr_buf
>
> Frazer Leslie Clews (2):
> fix format strings in makedevs.c in print statements
> fix invalidScanfFormatWidth to prevent overflowing usr_buf
Perhaps fixing these issues would be a good opportunity to align with
the Buildroot (ie upstream?) version instead of diverging even more.
https://git.buildroot.net/buildroot/tree/package/makedevs/makedevs.c
> meta/recipes-devtools/makedevs/makedevs/makedevs.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> --
> 2.20.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Bug fixes for makedevs.c
2019-10-28 19:11 ` [PATCH 0/2] " Andre McCurdy
@ 2019-10-28 21:22 ` Ross Burton
0 siblings, 0 replies; 6+ messages in thread
From: Ross Burton @ 2019-10-28 21:22 UTC (permalink / raw)
To: openembedded-core
On 28/10/2019 19:11, Andre McCurdy wrote:
> On Mon, Oct 28, 2019 at 10:15 AM Frazer Leslie Clews
> <frazer.clews@codethink.co.uk> wrote:
>>
>> fixes a few bugs in some print statements and an overflow bug with usr_buf
>>
>> Frazer Leslie Clews (2):
>> fix format strings in makedevs.c in print statements
>> fix invalidScanfFormatWidth to prevent overflowing usr_buf
>
> Perhaps fixing these issues would be a good opportunity to align with
> the Buildroot (ie upstream?) version instead of diverging even more.
>
> https://git.buildroot.net/buildroot/tree/package/makedevs/makedevs.c
Yes, this.
If we can just fetch makedevs from buildroot that's one less piece of
code we have to look after.
Ross
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-10-28 21:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-28 17:14 [PATCH 0/2] Bug fixes for makedevs.c Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 1/2] fix format strings in makedevs.c in print statements Frazer Leslie Clews
2019-10-28 17:14 ` [PATCH 2/2] fix invalidScanfFormatWidth to prevent overflowing usr_buf Frazer Leslie Clews
2019-10-28 17:32 ` ✗ patchtest: failure for Bug fixes for makedevs.c Patchwork
2019-10-28 19:11 ` [PATCH 0/2] " Andre McCurdy
2019-10-28 21:22 ` Ross Burton
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.