提交 15bded54 编写于 作者: W wanghao-free

fix: 回退错误合入的解决getgroups的list比设置的list多一个gid问题的修改

主干不存在该问题.

回退原因是要求维护这边合入的人没有跟问题修改人赵天宇确认,实际上这个问题单解决不在内核这边,
内核这边的提交需要回退

Close #I48FMK
Signed-off-by: Nwanghao-free <wanghao453@huawei.com>
上级 2a0db420
......@@ -683,12 +683,12 @@ int SysGetGroupID(void)
}
#ifdef LOSCFG_SECURITY_CAPABILITY
static int SetGroups(int size, const int *safeList)
static int SetGroups(int listSize, const int *safeList, int size)
{
User *oldUser = NULL;
unsigned int intSave;
User *newUser = LOS_MemAlloc(m_aucSysMem1, sizeof(User) + size * sizeof(int));
User *newUser = LOS_MemAlloc(m_aucSysMem1, sizeof(User) + listSize * sizeof(int));
if (newUser == NULL) {
return -ENOMEM;
}
......@@ -696,11 +696,14 @@ static int SetGroups(int size, const int *safeList)
SCHEDULER_LOCK(intSave);
oldUser = OsCurrUserGet();
(VOID)memcpy_s(newUser, sizeof(User), oldUser, sizeof(User));
if (size != 0) {
if (safeList != NULL) {
(VOID)memcpy_s(newUser->groups, size * sizeof(int), safeList, size * sizeof(int));
}
newUser->groupNumber = size;
if (listSize == size) {
newUser->groups[listSize] = oldUser->gid;
}
newUser->groupNumber = listSize + 1;
OsCurrProcessGet()->user = newUser;
SCHEDULER_UNLOCK(intSave);
......@@ -721,7 +724,7 @@ static int GetGroups(int size, int list[])
SCHEDULER_UNLOCK(intSave);
listSize = groupCount * sizeof(int);
if ((size == 0) || (groupCount == 0)) {
if (size == 0) {
return groupCount;
} else if (list == NULL) {
return -EFAULT;
......@@ -779,6 +782,7 @@ int SysSetGroups(int size, const int list[])
#ifdef LOSCFG_SECURITY_CAPABILITY
int ret;
int gid;
int listSize = size;
unsigned int count;
int *safeList = NULL;
#endif
......@@ -809,14 +813,16 @@ int SysSetGroups(int size, const int list[])
}
gid = OsCurrUserGet()->gid;
for (count = 0; count < size; count++) {
if (safeList[count] < 0) {
if (safeList[count] == gid) {
listSize = size - 1;
} else if (safeList[count] < 0) {
ret = -EINVAL;
goto EXIT;
}
}
}
ret = SetGroups(size, safeList);
ret = SetGroups(listSize, safeList, size);
EXIT:
if (safeList != NULL) {
(void)LOS_MemFree(m_aucSysMem1, safeList);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册