External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-07 17:28:15 +00:00
Code

conf: eliminate redundant use of VIR_ALLOC

Browse Source 
We can use VIR_REALLOC_N with NULL pointer, which behaves the same way
as VIR_ALLOC_N in that case, so no need for a condition that's
checking if some data are allocated already.

---

I tried to find other parts of the code similar to this, so I can do a
full cleanup for the whole repository, so I used this (excuse the long
line, but that's how I was writing it):

git grep -nHC 5 -e VIR_REALLOC_N -e VIR_ALLOC_N | while read line; do if [[ "$line" == "--" ]]; then if [[ ${#tmpbuf} -gt 10 && "$REALLOC_N" == "true" && "$ALLOC_N" == "true" ]]; then echo $line; while [[ ${#tmpbuf[*]} -gt 0 ]]; do echo "${tmpbuf[0]}"; tmpbuf=( "${tmpbuf[@]:1:${#tmpbuf[*]}}" ); done; fi; unset tmpbuf REALLOC_N ALLOC_N; else if [[ "$ALLOC_N" != "true" && "${line/VIR_ALLOC_N//}" != "${line}" ]]; then ALLOC_N="true"; fi; if [[ "$REALLOC_N" != "true" && "${line/VIR_REALLOC_N//}" != "${line}" ]]; then REALLOC_N="true"; fi; tmpbuf[${#tmpbuf[*]}]="$line"; fi; done | less

And reviewed the output just to find out this was the only occurrence of
the inconsistency.
This commit is contained in:
Martin Kletzander 2012-12-17 16:03:15 +01:00
parent 7affb25be9
commit b12a9cdedd
1 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/conf/domain_conf.c
View File

@ -9168,16 +9168,9 @@ static virDomainDefPtr virDomainDefParseXML(virCapsPtr caps,
* the policy specified explicitly as def->cpuset. * the policy specified explicitly as def->cpuset.
*/ */
if (def->cpumask) { if (def->cpumask) {
if (!def->cputune.vcpupin) { if (VIR_REALLOC_N(def->cputune.vcpupin, def->vcpus) < 0) {
if (VIR_ALLOC_N(def->cputune.vcpupin, def->vcpus) < 0) { virReportOOMError();
virReportOOMError(); goto error;
goto error;
}
} else {
if (VIR_REALLOC_N(def->cputune.vcpupin, def->vcpus) < 0) {
virReportOOMError();
goto error;
}
} }
for (i = 0; i < def->vcpus; i++) { for (i = 0; i < def->vcpus; i++) {