All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Anthony PERARD" <anthony.perard@vates.tech>
To: "Juergen Gross" <jgross@suse.com>
Cc: xen-devel@lists.xenproject.org, "Julien Grall" <julien@xen.org>
Subject: Re: [PATCH v2 02/12] tools/xenstored: add helper to parse domid
Date: Mon, 23 Mar 2026 12:30:01 +0000	[thread overview]
Message-ID: <acEySKHLW9beKXbS@l14> (raw)
In-Reply-To: <20260320150120.874878-3-jgross@suse.com>

On Fri, Mar 20, 2026 at 04:01:10PM +0100, Juergen Gross wrote:
> Today a domid passed in by a command is parsed using atoi(). This
> will still "succeed" even with a domid like "x", resulting in "0" to
> be used instead.
> 
> Use a common domid parser instead rejecting everything but integers
> in the range 0..65535 like specified in docs/misc/xenstore.txt.
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
> V2:
> - new patch (kind of suggested by Anthony Perard)

Reviewed-by: Anthony PERARD <anthony.perard@vates.tech>

Thanks,


--
Anthony Perard | Vates XCP-ng Developer

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech



  reply	other threads:[~2026-03-23 12:30 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-20 15:01 [PATCH v2 00/12] tools: add support for per-domain xenstore quota Juergen Gross
2026-03-20 15:01 ` [PATCH v2 01/12] tools/libs/store: add get- and set-quota related functions Juergen Gross
2026-03-23 12:16   ` Anthony PERARD
2026-03-20 15:01 ` [PATCH v2 02/12] tools/xenstored: add helper to parse domid Juergen Gross
2026-03-23 12:30   ` Anthony PERARD [this message]
2026-03-20 15:01 ` [PATCH v2 03/12] tools/xenstored: add central quota check functions Juergen Gross
2026-03-20 15:01 ` [PATCH v2 04/12] tools/xenstored: rework hard_quotas and soft_quotas arrays Juergen Gross
2026-03-20 15:01 ` [PATCH v2 05/12] tools/xenstored: add GLOBAL_QUOTA_DATA record for live update Juergen Gross
2026-03-23 12:50   ` Anthony PERARD
2026-03-20 15:01 ` [PATCH v2 06/12] tools/xenstored: split acc[] array in struct domain Juergen Gross
2026-03-20 15:01 ` [PATCH v2 07/12] tools/xenstored: use per-domain quota settings Juergen Gross
2026-03-20 15:01 ` [PATCH v2 08/12] tools/xenstored: implement the GET/SET_QUOTA commands Juergen Gross
2026-03-23 13:00   ` Anthony PERARD
2026-03-20 15:01 ` [PATCH v2 09/12] tools/libxl: add functions for retrieving and setting xenstore quota Juergen Gross
2026-03-23 13:19   ` Anthony PERARD
2026-03-20 15:01 ` [PATCH v2 10/12] tools/libxl: add support for xenstore quota in domain_config Juergen Gross
2026-03-23 13:21   ` Anthony PERARD
2026-03-20 15:01 ` [PATCH v2 11/12] tools/xl: add xl commands for xenstore quota operations Juergen Gross
2026-03-23 14:08   ` Anthony PERARD
2026-03-20 15:01 ` [PATCH v2 12/12] tools/xl: add support for xenstore quota setting via domain config Juergen Gross

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=acEySKHLW9beKXbS@l14 \
    --to=anthony.perard@vates.tech \
    --cc=jgross@suse.com \
    --cc=julien@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.