External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-02-21 19:02:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

util: xml: Reimplement virXPath(U)Int via virXPathEvalString

Browse Source 
Similarly to the refactor of virXPath(U)LongLong drop the ability to
convert from the internal double value forcing the use of the 'string()'
conversion.

In case of 32 bit integers there's no problem with overflows, but we can
implement the code identically to what we have in the other helpers.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
Peter Krempa 2022-10-05 13:04:17 +02:00
parent fe4120ec26
commit 90cb594cf6
1 changed files with 20 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/util/virxml.c
View File

@ -149,7 +149,9 @@ virXPathLongBase(const char *xpath,
* @ctxt: an XPath context
* @value: the returned int value
*
* Convenience function to evaluate an XPath number
* Convenience function to evaluate an XPath number. The @xpath expression
* must ensure that the evaluated value is returned as a string (use the
* 'string()' conversion in the expression).
*
* Returns 0 in case of success in which case @value is set,
* or -1 if the XPath evaluation failed or -2 if the
@ -160,15 +162,14 @@ virXPathInt(const char *xpath,
xmlXPathContextPtr ctxt,
int *value)
{
long tmp;
int ret;
g_autoptr(xmlXPathObject) obj = NULL;
ret = virXPathLongBase(xpath, ctxt, 10, &tmp);
if (ret < 0)
return ret;
if ((int) tmp != tmp)
if (!(obj = virXPathEvalString(xpath, ctxt)))
return -1;
if (virStrToLong_i((char *) obj->stringval, NULL, 10, value) < 0)
return -2;
*value = tmp;
return 0;
}
@ -230,28 +231,29 @@ virXPathULongBase(const char *xpath,
* virXPathUInt:
* @xpath: the XPath string to evaluate
* @ctxt: an XPath context
* @value: the returned int value
* @value: the returned unsigned int value
*
* Convenience function to evaluate an XPath number
* Convenience function to evaluate an XPath number. The @xpath expression
* must ensure that the evaluated value is returned as a string (use the
* 'string()' conversion in the expression).
*
* Returns 0 in case of success in which case @value is set,
* or -1 if the XPath evaluation failed or -2 if the
* value doesn't have an int format.
* value doesn't have an unsigned int format.
*/
int
virXPathUInt(const char *xpath,
xmlXPathContextPtr ctxt,
unsigned int *value)
{
unsigned long tmp;
int ret;
g_autoptr(xmlXPathObject) obj = NULL;
ret = virXPathULongBase(xpath, ctxt, 10, &tmp);
if (ret < 0)
return ret;
if ((unsigned int) tmp != tmp)
if (!(obj = virXPathEvalString(xpath, ctxt)))
return -1;
if (virStrToLong_ui((char *) obj->stringval, NULL, 10, value) < 0)
return -2;
*value = tmp;
return 0;
}