* [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing [not found] <cover.1704949719.git.pengfei.xu@intel.com> @ 2024-01-11 5:09 ` Pengfei Xu 2024-01-11 8:51 ` Petr Vorel 2024-01-11 10:09 ` Petr Vorel 0 siblings, 2 replies; 7+ messages in thread From: Pengfei Xu @ 2024-01-11 5:09 UTC (permalink / raw) To: ltp; +Cc: Heng Su, rpalethorpe Because the following patch will fix the add_key syscall incoming data boundary size issue, 6 bytes incoming data should pass: https://lore.kernel.org/all/1784441.1704907412@warthog.procyon.org.uk/ -> https://lore.kernel.org/all/1850031.1704921100@warthog.procyon.org.uk/ In order to test above boundary incoming data size, change the dns_res_payload size. Signed-off-by: Pengfei Xu <pengfei.xu@intel.com> --- testcases/kernel/syscalls/keyctl/keyctl05.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/testcases/kernel/syscalls/keyctl/keyctl05.c b/testcases/kernel/syscalls/keyctl/keyctl05.c index 90cf1ef06..50e78853c 100644 --- a/testcases/kernel/syscalls/keyctl/keyctl05.c +++ b/testcases/kernel/syscalls/keyctl/keyctl05.c @@ -199,11 +199,11 @@ static void do_test(unsigned int i) * We need to pass check in dns_resolver_preparse(), * give it dummy server list request. * From v6.7-rc8 commit 1997b3cb4217b09e49659b634c94da47f0340409: - * the incoming data for add_key syscall should be larger than 6 bytes, - * because struct dns_server_list_v1_header without body is 6 bytes. + * the incoming data for add_key syscall should be not less than 6 + * bytes, because struct dns_server_list_v1_header is 6 bytes. + * The minimum payload will be tested here for boundary testing. */ - static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, \ - 0x00, 0x00 }; + static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, 0x00 }; switch (i) { case 0: -- 2.39.3 -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing 2024-01-11 5:09 ` [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing Pengfei Xu @ 2024-01-11 8:51 ` Petr Vorel 2024-01-11 9:12 ` Pengfei Xu 2024-01-11 10:09 ` Petr Vorel 1 sibling, 1 reply; 7+ messages in thread From: Petr Vorel @ 2024-01-11 8:51 UTC (permalink / raw) To: Pengfei Xu; +Cc: Heng Su, ltp > Because the following patch will fix the add_key syscall incoming data boundary > size issue, 6 bytes incoming data should pass: > https://lore.kernel.org/all/1784441.1704907412@warthog.procyon.org.uk/ > -> https://lore.kernel.org/all/1850031.1704921100@warthog.procyon.org.uk/ > In order to test above boundary incoming data size, change the dns_res_payload > size. > Signed-off-by: Pengfei Xu <pengfei.xu@intel.com> > --- > testcases/kernel/syscalls/keyctl/keyctl05.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > diff --git a/testcases/kernel/syscalls/keyctl/keyctl05.c b/testcases/kernel/syscalls/keyctl/keyctl05.c > index 90cf1ef06..50e78853c 100644 > --- a/testcases/kernel/syscalls/keyctl/keyctl05.c > +++ b/testcases/kernel/syscalls/keyctl/keyctl05.c > @@ -199,11 +199,11 @@ static void do_test(unsigned int i) > * We need to pass check in dns_resolver_preparse(), > * give it dummy server list request. > * From v6.7-rc8 commit 1997b3cb4217b09e49659b634c94da47f0340409: > - * the incoming data for add_key syscall should be larger than 6 bytes, > - * because struct dns_server_list_v1_header without body is 6 bytes. > + * the incoming data for add_key syscall should be not less than 6 > + * bytes, because struct dns_server_list_v1_header is 6 bytes. > + * The minimum payload will be tested here for boundary testing. > */ > - static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, \ nit (already pushed): \ is not needed for arrays. Kind regards, Petr > - 0x00, 0x00 }; > + static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, 0x00 }; > switch (i) { > case 0: -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing 2024-01-11 8:51 ` Petr Vorel @ 2024-01-11 9:12 ` Pengfei Xu 0 siblings, 0 replies; 7+ messages in thread From: Pengfei Xu @ 2024-01-11 9:12 UTC (permalink / raw) To: Petr Vorel; +Cc: Heng Su, ltp On 2024-01-11 at 09:51:52 +0100, Petr Vorel wrote: > > Because the following patch will fix the add_key syscall incoming data boundary > > size issue, 6 bytes incoming data should pass: > > https://lore.kernel.org/all/1784441.1704907412@warthog.procyon.org.uk/ > > -> https://lore.kernel.org/all/1850031.1704921100@warthog.procyon.org.uk/ > > In order to test above boundary incoming data size, change the dns_res_payload > > size. > > > Signed-off-by: Pengfei Xu <pengfei.xu@intel.com> > > --- > > testcases/kernel/syscalls/keyctl/keyctl05.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > diff --git a/testcases/kernel/syscalls/keyctl/keyctl05.c b/testcases/kernel/syscalls/keyctl/keyctl05.c > > index 90cf1ef06..50e78853c 100644 > > --- a/testcases/kernel/syscalls/keyctl/keyctl05.c > > +++ b/testcases/kernel/syscalls/keyctl/keyctl05.c > > @@ -199,11 +199,11 @@ static void do_test(unsigned int i) > > * We need to pass check in dns_resolver_preparse(), > > * give it dummy server list request. > > * From v6.7-rc8 commit 1997b3cb4217b09e49659b634c94da47f0340409: > > - * the incoming data for add_key syscall should be larger than 6 bytes, > > - * because struct dns_server_list_v1_header without body is 6 bytes. > > + * the incoming data for add_key syscall should be not less than 6 > > + * bytes, because struct dns_server_list_v1_header is 6 bytes. > > + * The minimum payload will be tested here for boundary testing. > > */ > > - static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, \ > > nit (already pushed): \ is not needed for arrays. Indeed, thanks for your remind! Thanks! > > Kind regards, > Petr > > > - 0x00, 0x00 }; > > + static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, 0x00 }; > > > switch (i) { > > case 0: -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing 2024-01-11 5:09 ` [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing Pengfei Xu 2024-01-11 8:51 ` Petr Vorel @ 2024-01-11 10:09 ` Petr Vorel 2024-01-11 12:24 ` Pengfei Xu 1 sibling, 1 reply; 7+ messages in thread From: Petr Vorel @ 2024-01-11 10:09 UTC (permalink / raw) To: Pengfei Xu; +Cc: Heng Su, ltp Hi Xu, > Because the following patch will fix the add_key syscall incoming data boundary > size issue, 6 bytes incoming data should pass: > https://lore.kernel.org/all/1784441.1704907412@warthog.procyon.org.uk/ > -> https://lore.kernel.org/all/1850031.1704921100@warthog.procyon.org.uk/ > In order to test above boundary incoming data size, change the dns_res_payload > size. I'm sorry, I was faster and just fixed \ myself. Rebasing is trivial, but there was missing commit (already merged and other things). Thus I have sent v2 under your name (which I actually forget to mark as v2 :(). https://lore.kernel.org/ltp/20240111100636.1897637-1-pvorel@suse.cz/ https://patchwork.ozlabs.org/project/ltp/patch/20240111100636.1897637-1-pvorel@suse.cz/ Can you please have a look on it? Kind regards, Petr > Signed-off-by: Pengfei Xu <pengfei.xu@intel.com> > --- > testcases/kernel/syscalls/keyctl/keyctl05.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > diff --git a/testcases/kernel/syscalls/keyctl/keyctl05.c b/testcases/kernel/syscalls/keyctl/keyctl05.c > index 90cf1ef06..50e78853c 100644 > --- a/testcases/kernel/syscalls/keyctl/keyctl05.c > +++ b/testcases/kernel/syscalls/keyctl/keyctl05.c > @@ -199,11 +199,11 @@ static void do_test(unsigned int i) > * We need to pass check in dns_resolver_preparse(), > * give it dummy server list request. > * From v6.7-rc8 commit 1997b3cb4217b09e49659b634c94da47f0340409: > - * the incoming data for add_key syscall should be larger than 6 bytes, > - * because struct dns_server_list_v1_header without body is 6 bytes. > + * the incoming data for add_key syscall should be not less than 6 > + * bytes, because struct dns_server_list_v1_header is 6 bytes. > + * The minimum payload will be tested here for boundary testing. > */ > - static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, \ > - 0x00, 0x00 }; > + static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, 0x00 }; > switch (i) { > case 0: -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing 2024-01-11 10:09 ` Petr Vorel @ 2024-01-11 12:24 ` Pengfei Xu 2024-01-22 18:22 ` Petr Vorel 0 siblings, 1 reply; 7+ messages in thread From: Pengfei Xu @ 2024-01-11 12:24 UTC (permalink / raw) To: Petr Vorel; +Cc: Heng Su, ltp Hi Petr, On 2024-01-11 at 11:09:49 +0100, Petr Vorel wrote: > Hi Xu, > > > Because the following patch will fix the add_key syscall incoming data boundary > > size issue, 6 bytes incoming data should pass: > > https://lore.kernel.org/all/1784441.1704907412@warthog.procyon.org.uk/ > > -> https://lore.kernel.org/all/1850031.1704921100@warthog.procyon.org.uk/ > > In order to test above boundary incoming data size, change the dns_res_payload > > size. > > I'm sorry, I was faster and just fixed \ myself. > Rebasing is trivial, but there was missing commit (already merged and other > things). Thus I have sent v2 under your name (which I actually forget to mark > as v2 :(). > > https://lore.kernel.org/ltp/20240111100636.1897637-1-pvorel@suse.cz/ > https://patchwork.ozlabs.org/project/ltp/patch/20240111100636.1897637-1-pvorel@suse.cz/ > > Can you please have a look on it? Yes, it's better to add the linux-git acc657692aed and it looks good. Thanks for your improvment! Best Regards, Thanks! > > Kind regards, > Petr > > > Signed-off-by: Pengfei Xu <pengfei.xu@intel.com> > > --- > > testcases/kernel/syscalls/keyctl/keyctl05.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > diff --git a/testcases/kernel/syscalls/keyctl/keyctl05.c b/testcases/kernel/syscalls/keyctl/keyctl05.c > > index 90cf1ef06..50e78853c 100644 > > --- a/testcases/kernel/syscalls/keyctl/keyctl05.c > > +++ b/testcases/kernel/syscalls/keyctl/keyctl05.c > > @@ -199,11 +199,11 @@ static void do_test(unsigned int i) > > * We need to pass check in dns_resolver_preparse(), > > * give it dummy server list request. > > * From v6.7-rc8 commit 1997b3cb4217b09e49659b634c94da47f0340409: > > - * the incoming data for add_key syscall should be larger than 6 bytes, > > - * because struct dns_server_list_v1_header without body is 6 bytes. > > + * the incoming data for add_key syscall should be not less than 6 > > + * bytes, because struct dns_server_list_v1_header is 6 bytes. > > + * The minimum payload will be tested here for boundary testing. > > */ > > - static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, \ > > - 0x00, 0x00 }; > > + static char dns_res_payload[] = { 0x00, 0x00, 0x01, 0xff, 0x00, 0x00 }; > > > switch (i) { > > case 0: -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing 2024-01-11 12:24 ` Pengfei Xu @ 2024-01-22 18:22 ` Petr Vorel 2024-01-23 1:23 ` Pengfei Xu 0 siblings, 1 reply; 7+ messages in thread From: Petr Vorel @ 2024-01-22 18:22 UTC (permalink / raw) To: Pengfei Xu; +Cc: Heng Su, ltp Hi Xu, all, fix will be backported to stable series: 6.7, 6.6, 6.1, 5.15 and 5.10, it's not applicable to 4.19 and 5.4 stable). Therefore merged. Thanks! Kind regards, Petr -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing 2024-01-22 18:22 ` Petr Vorel @ 2024-01-23 1:23 ` Pengfei Xu 0 siblings, 0 replies; 7+ messages in thread From: Pengfei Xu @ 2024-01-23 1:23 UTC (permalink / raw) To: Petr Vorel; +Cc: Heng Su, ltp Hi Petr, On 2024-01-22 at 19:22:28 +0100, Petr Vorel wrote: > Hi Xu, all, > > fix will be backported to stable series: 6.7, 6.6, 6.1, 5.15 and 5.10, > it's not applicable to 4.19 and 5.4 stable). > > Therefore merged. Thanks! Great, thank you for your info and remind in community: https://lore.kernel.org/all/CA+icZUUc_0M_6JU3dZzVqrUUrWJceY1uD8dO2yFMCwtHtkaa_Q@mail.gmail.com/ Best Regards, Thanks! > > Kind regards, > Petr -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-01-23 1:29 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1704949719.git.pengfei.xu@intel.com>
2024-01-11 5:09 ` [LTP] [PATCH v1 1/1] keyctl05: improve the dns_res_payload for boundary testing Pengfei Xu
2024-01-11 8:51 ` Petr Vorel
2024-01-11 9:12 ` Pengfei Xu
2024-01-11 10:09 ` Petr Vorel
2024-01-11 12:24 ` Pengfei Xu
2024-01-22 18:22 ` Petr Vorel
2024-01-23 1:23 ` Pengfei Xu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox