[PATCH v3 0/2] qga: Ditch g_get_host_name()

Michal Privoznik posted 2 patches 3 years, 9 months ago
Test asan passed
Test FreeBSD passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1592849834.git.mprivozn@redhat.com
Maintainers: Stefan Weil <sw@weilnetz.de>, Michael Roth <mdroth@linux.vnet.ibm.com>, Paolo Bonzini <pbonzini@redhat.com>
include/qemu/osdep.h | 10 ++++++++++
qga/commands.c       | 17 +++++++++++++----
util/oslib-posix.c   | 35 +++++++++++++++++++++++++++++++++++
util/oslib-win32.c   | 13 +++++++++++++
4 files changed, 71 insertions(+), 4 deletions(-)
[PATCH v3 0/2] qga: Ditch g_get_host_name()
Posted by Michal Privoznik 3 years, 9 months ago
v3 of:

https://lists.nongnu.org/archive/html/qemu-devel/2020-06/msg06913.html

diff to v2:
- don't leak @hostname in util/oslib-posix.c:qemu_get_host_name()
- document why we are allocating one byte more than needed
- switch to g_new0() from g_malloc0().

Michal Privoznik (2):
  util: Introduce qemu_get_host_name()
  qga: Use qemu_get_host_name() instead of g_get_host_name()

 include/qemu/osdep.h | 10 ++++++++++
 qga/commands.c       | 17 +++++++++++++----
 util/oslib-posix.c   | 35 +++++++++++++++++++++++++++++++++++
 util/oslib-win32.c   | 13 +++++++++++++
 4 files changed, 71 insertions(+), 4 deletions(-)

-- 
2.26.2


Re: [PATCH v3 0/2] qga: Ditch g_get_host_name()
Posted by Michal Privoznik 3 years, 8 months ago
On 6/22/20 8:19 PM, Michal Privoznik wrote:
> v3 of:
> 
> https://lists.nongnu.org/archive/html/qemu-devel/2020-06/msg06913.html
> 
> diff to v2:
> - don't leak @hostname in util/oslib-posix.c:qemu_get_host_name()
> - document why we are allocating one byte more than needed
> - switch to g_new0() from g_malloc0().
> 
> Michal Privoznik (2):
>    util: Introduce qemu_get_host_name()
>    qga: Use qemu_get_host_name() instead of g_get_host_name()
> 
>   include/qemu/osdep.h | 10 ++++++++++
>   qga/commands.c       | 17 +++++++++++++----
>   util/oslib-posix.c   | 35 +++++++++++++++++++++++++++++++++++
>   util/oslib-win32.c   | 13 +++++++++++++
>   4 files changed, 71 insertions(+), 4 deletions(-)
> 

Ping? How can I get these merged please?

Michal


Re: [PATCH v3 0/2] qga: Ditch g_get_host_name()
Posted by Philippe Mathieu-Daudé 3 years, 8 months ago
Ping^2?

On 7/10/20 11:43 AM, Michal Privoznik wrote:
> On 6/22/20 8:19 PM, Michal Privoznik wrote:
>> v3 of:
>>
>> https://lists.nongnu.org/archive/html/qemu-devel/2020-06/msg06913.html
>>
>> diff to v2:
>> - don't leak @hostname in util/oslib-posix.c:qemu_get_host_name()
>> - document why we are allocating one byte more than needed
>> - switch to g_new0() from g_malloc0().
>>
>> Michal Privoznik (2):
>>    util: Introduce qemu_get_host_name()
>>    qga: Use qemu_get_host_name() instead of g_get_host_name()
>>
>>   include/qemu/osdep.h | 10 ++++++++++
>>   qga/commands.c       | 17 +++++++++++++----
>>   util/oslib-posix.c   | 35 +++++++++++++++++++++++++++++++++++
>>   util/oslib-win32.c   | 13 +++++++++++++
>>   4 files changed, 71 insertions(+), 4 deletions(-)
>>
> 
> Ping? How can I get these merged please?
> 
> Michal
> 
>