From e225ec8375941d6e596cca4fb197cf39a722b40c Mon Sep 17 00:00:00 2001 From: Per Cederqvist <ceder@lysator.liu.se> Date: Sat, 29 Sep 2001 20:11:05 +0000 Subject: [PATCH] (create_person_generic): This can leak conference numbers. Write a comment about it. --- src/server/person.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/server/person.c b/src/server/person.c index 2cf49c5ae..f82bd897d 100644 --- a/src/server/person.c +++ b/src/server/person.c @@ -1,5 +1,5 @@ /* - * $Id: person.c,v 0.59 1999/07/10 22:38:41 ceder Exp $ + * $Id: person.c,v 0.60 2001/09/29 20:11:05 ceder Exp $ * Copyright (C) 1991-1999 Lysator Academic Computer Association. * * This file is part of the LysKOM server. @@ -35,7 +35,7 @@ #endif static const char * -rcsid = "$Id: person.c,v 0.59 1999/07/10 22:38:41 ceder Exp $"; +rcsid = "$Id: person.c,v 0.60 2001/09/29 20:11:05 ceder Exp $"; #include "rcs.h" USE(rcsid); @@ -670,6 +670,11 @@ create_person_generic ( const String name, new_user, TRUE) != OK) { + /* FIXME: Conf_no leak: We create a person, check the + aux-items, determine that the aux-items are bogus, and + immediately delete the person. This leaks a conference + number. We can live with that, but it isn't pretty. */ + cached_delete_conf(new_user); cached_delete_person(new_user); -- GitLab