sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

mm/iob: Don't return NULL in iob_pack

Browse Source 
We don't want to get a NULL pointer after iob_pack on an IOB chain with
several iobs with length 0, it should return one IOB with length 0.
Otherwise each place calls iob_pack needs to check the result.

Signed-off-by: Zhe Weng <wengzhe@xiaomi.com>
This commit is contained in:
Zhe Weng 2023-04-20 12:05:17 +08:00 committed by Xiang Xiao
parent 1aceb1d872
commit 44a04733d4
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
mm/iob/iob_pack.c
View File

@ -51,15 +51,11 @@ FAR struct iob_s *iob_pack(FAR struct iob_s *iob)
unsigned int ncopy; unsigned int ncopy;
unsigned int navail; unsigned int navail;
/* Handle special cases */ /* Handle special cases, preserve at least one iob. */
while (iob->io_len <= 0) while (iob->io_len <= 0 && iob->io_flink != NULL)
{ {
iob = iob_free(iob); iob = iob_free(iob);
if (iob == NULL)
{
return NULL;
}
} }
/* Now remember the head of the chain (for the return value) */ /* Now remember the head of the chain (for the return value) */