hv: refine strncpy_s to only one exit point
Fix procedure has more than one exit point Tracked-On: #861 Signed-off-by: Li, Fei1 <fei1.li@intel.com> Acked-by: Eddie Dong <eddie.dong@intel.com> Reviewed-by: Huang, Yonghua <yonghua.huang@intel.com>
This commit is contained in:
parent
b8ca17c6da
commit
2f30dcdb37
|
@ -203,63 +203,68 @@ char *strncpy_s(char *d_arg, size_t dmax, const char *s_arg, size_t slen_arg)
|
||||||
{
|
{
|
||||||
const char *s = s_arg;
|
const char *s = s_arg;
|
||||||
char *d = d_arg;
|
char *d = d_arg;
|
||||||
char *dest_base;
|
char *pret;
|
||||||
size_t dest_avail;
|
size_t dest_avail;
|
||||||
uint64_t overlap_guard;
|
uint64_t overlap_guard;
|
||||||
size_t slen = slen_arg;
|
size_t slen = slen_arg;
|
||||||
|
|
||||||
if ((d == NULL) || (s == NULL)) {
|
if ((d == NULL) || (s == NULL)) {
|
||||||
pr_err("%s: invlaid src or dest buffer", __func__);
|
pr_err("%s: invlaid src or dest buffer", __func__);
|
||||||
return NULL;
|
pret = NULL;
|
||||||
|
} else {
|
||||||
|
pret = d_arg;
|
||||||
}
|
}
|
||||||
|
|
||||||
if ((dmax == 0U) || (slen == 0U)) {
|
if (pret != NULL) {
|
||||||
pr_err("%s: invlaid length of src or dest buffer", __func__);
|
if ((dmax == 0U) || (slen == 0U)) {
|
||||||
return NULL;
|
pr_err("%s: invlaid length of src or dest buffer", __func__);
|
||||||
|
pret = NULL;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (d == s) {
|
/* if d equal to s, just return d; else execute the below code */
|
||||||
return d;
|
if ((pret != NULL) && (d != s)) {
|
||||||
}
|
overlap_guard = (uint64_t)((d > s) ? (d - s - 1) : (s - d - 1));
|
||||||
|
dest_avail = dmax;
|
||||||
|
|
||||||
overlap_guard = (uint64_t)((d > s) ? (d - s - 1) : (s - d - 1));
|
while (dest_avail > 0U) {
|
||||||
|
if (overlap_guard == 0U) {
|
||||||
|
pr_err("%s: overlap happened.", __func__);
|
||||||
|
d--;
|
||||||
|
*d = '\0';
|
||||||
|
/* break out to return */
|
||||||
|
pret = NULL;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
dest_base = d;
|
if (slen == 0U) {
|
||||||
dest_avail = dmax;
|
*d = '\0';
|
||||||
|
/* break out to return */
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
while (dest_avail > 0U) {
|
*d = *s;
|
||||||
if (overlap_guard == 0U) {
|
if (*d == '\0') {
|
||||||
pr_err("%s: overlap happened.", __func__);
|
/* break out to return */
|
||||||
d--;
|
break;
|
||||||
*d = '\0';
|
}
|
||||||
return NULL;
|
|
||||||
|
d++;
|
||||||
|
s++;
|
||||||
|
slen--;
|
||||||
|
dest_avail--;
|
||||||
|
overlap_guard--;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (slen == 0U) {
|
if (dest_avail == 0U) {
|
||||||
*d = '\0';
|
pr_err("%s: dest buffer has no enough space.", __func__);
|
||||||
return dest_base;
|
|
||||||
}
|
|
||||||
|
|
||||||
*d = *s;
|
/* to avoid a string that is not null-terminated in dest buffer */
|
||||||
if (*d == '\0') {
|
pret[dmax - 1] = '\0';
|
||||||
return dest_base;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
d++;
|
|
||||||
s++;
|
|
||||||
slen--;
|
|
||||||
dest_avail--;
|
|
||||||
overlap_guard--;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
pr_err("%s: dest buffer has no enough space.", __func__);
|
return pret;
|
||||||
|
|
||||||
/*
|
|
||||||
* to avoid a string that is not
|
|
||||||
* null-terminated in dest buffer
|
|
||||||
*/
|
|
||||||
dest_base[dmax - 1] = '\0';
|
|
||||||
return NULL;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in New Issue