On Mon, Jun 14, 2021 at 11:30:12AM +0200, sieberf@xxxxxxxxxx wrote: > + new_dir = debugfs_create_dir(name, root); > + dbgfs_dirs[dbgfs_nr_ctxs] = new_dir; > > debugfs_create_dir might return ERR_PTR, should this be checked? No. Why do you care, the logic will keep on working afterward even if that were to happen. thanks, greg k-h