[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 02/19] python/qmp: Fix type of SocketAddrT
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v3 02/19] python/qmp: Fix type of SocketAddrT |
Date: |
Thu, 3 Jun 2021 09:07:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
On 6/3/21 2:37 AM, John Snow wrote:
> In porting the qom tools, qmp-shell, etc; it becomes evident that this
> type is wrong.
>
> This is an integer, not a string. We didn't catch this before because
> none of QEMUMonitorProtocol's *users* happen to be checked, and the
> internal logic of this class is otherwise self-consistent. Additionally,
> mypy was not introspecting into the socket() interface to realize we
> were passing a bad type for AF_INET. Fixed now.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> python/qemu/qmp/__init__.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
- [PATCH v3 00/19] Python: move /scripts/qmp/qom* to /python/qemu/qmp/qom*, John Snow, 2021/06/02
- [PATCH v3 03/19] python/qmp: add parse_address classmethod, John Snow, 2021/06/02
- [PATCH v3 02/19] python/qmp: Fix type of SocketAddrT, John Snow, 2021/06/02
- Re: [PATCH v3 02/19] python/qmp: Fix type of SocketAddrT,
Philippe Mathieu-Daudé <=
- [PATCH v3 01/19] python/pipenv: Update Pipfile.lock, John Snow, 2021/06/02
- [PATCH v3 05/19] python/qmp: add qom script entry points, John Snow, 2021/06/02
- [PATCH v3 04/19] python/qmp: Add qom script rewrites, John Snow, 2021/06/02
- [PATCH v3 06/19] scripts/qmp: redirect qom-xxx scripts to python/qemu/qmp/, John Snow, 2021/06/02
- [PATCH v3 07/19] scripts/qom-fuse: apply isort rules, John Snow, 2021/06/02
- [PATCH v3 08/19] scripts/qom-fuse: apply flake8 rules, John Snow, 2021/06/02
- [PATCH v3 09/19] python: Add 'fh' to known-good variable names, John Snow, 2021/06/02
- [PATCH v3 10/19] scripts/qom-fuse: Apply pylint rules, John Snow, 2021/06/02
- [PATCH v3 11/19] scripts/qom-fuse: Add docstrings, John Snow, 2021/06/02