diff options
| author | Dimitri Staessens <dimitri@ouroboros.rocks> | 2020-05-28 17:07:30 +0200 |
|---|---|---|
| committer | Sander Vrijders <sander@ouroboros.rocks> | 2020-05-29 11:13:31 +0200 |
| commit | 5c12e2efc066f09c0e7ed21ed0bb3a3ce635cb0a (patch) | |
| tree | f05fa585164f8ae932fd6ed7e1a23b33c9ead69a | |
| parent | d68bb3b678e867c69a99f97e26b72715e2b8e57f (diff) | |
| download | ouroboros-5c12e2efc066f09c0e7ed21ed0bb3a3ce635cb0a.tar.gz ouroboros-5c12e2efc066f09c0e7ed21ed0bb3a3ce635cb0a.zip | |
ipcpd: Fix bad index in packet scheduler
GCC 10 static analyzer found that the wrong index was used in the fail
path of psched_create, causing double (multiple) frees.
Signed-off-by: Dimitri Staessens <dimitri@ouroboros.rocks>
Signed-off-by: Sander Vrijders <sander@ouroboros.rocks>
| -rw-r--r-- | src/ipcpd/unicast/psched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/ipcpd/unicast/psched.c b/src/ipcpd/unicast/psched.c index 1ac3fc12..898e6b27 100644 --- a/src/ipcpd/unicast/psched.c +++ b/src/ipcpd/unicast/psched.c @@ -160,7 +160,7 @@ struct psched * psched_create(next_packet_fn_t callback) for (j = 0; j < i; ++j) pthread_join(psched->readers[j], NULL); for (j = i; j < QOS_CUBE_MAX * IPCP_SCHED_THR_MUL; ++j) - free(infos[i]); + free(infos[j]); goto fail_infos; } } |