提交 57c008f8 编写于 作者: M Martin Kletzander

conf: Fix invalid condition when parsing storage owner

In commit d2632d60 we agreed taht we want the parsed uid to properly
overflow but only to -1, however the value was read into long and then
wrapped into uid_t.  That meaned it failed on 32-bit systems.
Signed-off-by: NMartin Kletzander <mkletzan@redhat.com>
上级 18f03166
...@@ -734,7 +734,7 @@ virStorageDefParsePerms(xmlXPathContextPtr ctxt, ...@@ -734,7 +734,7 @@ virStorageDefParsePerms(xmlXPathContextPtr ctxt,
int defaultmode) int defaultmode)
{ {
char *mode; char *mode;
long val; long long val;
int ret = -1; int ret = -1;
xmlNodePtr relnode; xmlNodePtr relnode;
xmlNodePtr node; xmlNodePtr node;
...@@ -771,7 +771,7 @@ virStorageDefParsePerms(xmlXPathContextPtr ctxt, ...@@ -771,7 +771,7 @@ virStorageDefParsePerms(xmlXPathContextPtr ctxt,
if (virXPathNode("./owner", ctxt) == NULL) { if (virXPathNode("./owner", ctxt) == NULL) {
perms->uid = (uid_t) -1; perms->uid = (uid_t) -1;
} else { } else {
if (virXPathLong("number(./owner)", ctxt, &val) < 0 || if (virXPathLongLong("number(./owner)", ctxt, &val) < 0 ||
((uid_t)val != val && ((uid_t)val != val &&
val != -1)) { val != -1)) {
virReportError(VIR_ERR_XML_ERROR, "%s", virReportError(VIR_ERR_XML_ERROR, "%s",
...@@ -785,7 +785,7 @@ virStorageDefParsePerms(xmlXPathContextPtr ctxt, ...@@ -785,7 +785,7 @@ virStorageDefParsePerms(xmlXPathContextPtr ctxt,
if (virXPathNode("./group", ctxt) == NULL) { if (virXPathNode("./group", ctxt) == NULL) {
perms->gid = (gid_t) -1; perms->gid = (gid_t) -1;
} else { } else {
if (virXPathLong("number(./group)", ctxt, &val) < 0 || if (virXPathLongLong("number(./group)", ctxt, &val) < 0 ||
((gid_t) val != val && ((gid_t) val != val &&
val != -1)) { val != -1)) {
virReportError(VIR_ERR_XML_ERROR, "%s", virReportError(VIR_ERR_XML_ERROR, "%s",
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册