[PATCH 0/8] tools: vsh: Fix validation of command parameter lists

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Recent patch:

https://www.redhat.com/archives/libvir-list/2020-November/msg00555.html

passes build check but then virsh complains when started that the
command list is wrong.

Fix it by moving the check to the appropriate place.

You can use the above patch as reproducer.

Peter Krempa (8):
  tools: vsh: Unexport vshCmddefHelp
  tools: vshCmddefCheckInternals: Add parameter name to error message
  tools: vshCmddefCheckInternals: Port mandatory options check from
    vshCmddefOptParse
  tools: cmdSelfTest: Drop misleading comment
  tools: vshCmddefOptParse: Remove 'optional' command validation
  tools: vshCmddefHelp: Don't call vshCmddefOptParse
  tools: vshCmddefOptParse: Remove return value
  tools: virsh: Reset error when keepalive registration fails

 tools/virsh.c |   1 +
 tools/vsh.c   | 108 +++++++++++++++++++++-----------------------------
 tools/vsh.h   |   1 -
 3 files changed, 47 insertions(+), 63 deletions(-)

-- 
2.28.0




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux