=================================================================== RCS file: /cvsrepo/anoncvs/cvs/src/usr.bin/cvs/tag.c,v retrieving revision 1.83 retrieving revision 1.84 diff -c -r1.83 -r1.84 *** src/usr.bin/cvs/tag.c 2015/12/22 21:36:57 1.83 --- src/usr.bin/cvs/tag.c 2016/10/13 20:51:25 1.84 *************** *** 1,4 **** ! /* $OpenBSD: tag.c,v 1.83 2015/12/22 21:36:57 mmcc Exp $ */ /* * Copyright (c) 2006 Xavier Santolaria * --- 1,4 ---- ! /* $OpenBSD: tag.c,v 1.84 2016/10/13 20:51:25 fcambus Exp $ */ /* * Copyright (c) 2006 Xavier Santolaria * *************** *** 285,291 **** goto bad; rcsnum_tostr(rev, rbuf, sizeof(rbuf)); fi->crevstr = xstrdup(rbuf); ! rcsnum_free(rev); } else if (runflags & T_DELETE) goto bad; --- 285,291 ---- goto bad; rcsnum_tostr(rev, rbuf, sizeof(rbuf)); fi->crevstr = xstrdup(rbuf); ! free(rev); } else if (runflags & T_DELETE) goto bad; *************** *** 314,321 **** free(fi->nrevstr); free(fi->tag_new); free(fi->tag_old); ! if (rev != NULL) ! rcsnum_free(rev); free(fi); } --- 314,320 ---- free(fi->nrevstr); free(fi->tag_new); free(fi->tag_old); ! free(rev); free(fi); } *************** *** 425,435 **** trev = rcs_sym_getrev(cf->file_rcs, tag_name); if (trev != NULL) { if (rcsnum_cmp(srev, trev, 0) == 0) { ! rcsnum_free(trev); return (-1); } (void)rcsnum_tostr(trev, trevbuf, sizeof(trevbuf)); ! rcsnum_free(trev); if (!(runflags & T_FORCE_MOVE)) { cvs_printf("W %s : %s ", cf->file_path, tag_name); --- 424,434 ---- trev = rcs_sym_getrev(cf->file_rcs, tag_name); if (trev != NULL) { if (rcsnum_cmp(srev, trev, 0) == 0) { ! free(trev); return (-1); } (void)rcsnum_tostr(trev, trevbuf, sizeof(trevbuf)); ! free(trev); if (!(runflags & T_FORCE_MOVE)) { cvs_printf("W %s : %s ", cf->file_path, tag_name); *************** *** 460,469 **** "failed to set tag %s to revision %s in %s", tag_name, revbuf, cf->file_rcs->rf_path); } ! rcsnum_free(trev); return (-1); } ! rcsnum_free(trev); return (0); } --- 459,468 ---- "failed to set tag %s to revision %s in %s", tag_name, revbuf, cf->file_rcs->rf_path); } ! free(trev); return (-1); } ! free(trev); return (0); }