[iproute2,1/2] devlink: do dry parse for extended handle with selector

Commit Message

Jiri Pirko Nov. 26, 2024, 9:08 a.m. UTC

From: Jiri Pirko <[e-mail]>

When parsing with selector, there's a list of extended handles
(devname/busname/x) which require special treatment.
DL_OPT_HANDLEP is one of them. The code tries to parse devname/busname
handle and in case it is successful, it goes the "dump" way. However if
it's not, parsing is directly done. That is wrong, as the options may
still be incomplete. Do break in that case instead allowing to do dry
parse and possibly go the "dump" way in case the option list is not
complete.

Fixes: 70faecdca8f5 ("devlink: implement dump selector for devlink objects show commands")
Signed-off-by: Jiri Pirko <[e-mail]>
---
 devlink/devlink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

diff --git a/devlink/devlink.c b/devlink/devlink.c
index 9907712e3ad0..f4e90b804fb6 100644
--- a/devlink/devlink.c
+++ b/devlink/devlink.c
@@ -2409,7 +2409,7 @@  static int dl_argv_parse_with_selector(struct dl *dl, uint16_t *flags,
 						o_optional);
 			if (err == -ENOENT || !err)
 				goto dump_parse;
-			goto do_parse;
+			break;
 		}
 	}