tags 630485 +patch
thanks
Hi,
We are a group of women learning how to package and fix bug for debian and
we have prepared a patch for msort using the fix proposed in last mail from
marga. While working on the issue we found a problem with clean target so we
fixed that as well.
We attach the patch with both fixes.
Regards,
Lucia Prado, Julia Palandri, Romanella Di Ferdinando
diff -u msort-8.52/debian/changelog msort-8.52/debian/changelog
--- msort-8.52/debian/changelog
+++ msort-8.52/debian/changelog
@@ -1,3 +1,13 @@
+msort (8.52-1.3) unstable; urgency=low
+
+ * Non-maintainer upload.
+ * Fixed segfault when checking length of dynamic string in dstr.c
+ (Closes: #630485)
+ * Fixed debian/rules to clean properly. Delete msg and configure-stamp
+ files and distclean instead of clean.
+
+ -- Romanella Di Ferdinando <romanell...@gmail.com> Sat, 18 Jun 2011 12:06:27 -0300
+
msort (8.52-1.2) unstable; urgency=medium
* Non-maintainer upload.
diff -u msort-8.52/debian/rules msort-8.52/debian/rules
--- msort-8.52/debian/rules
+++ msort-8.52/debian/rules
@@ -37,8 +37,8 @@
clean:
dh_testdir
dh_testroot
- rm -f build-arch-stamp build-indep-stamp #CONFIGURE-STAMP#
- [ ! -f Makefile ] || $(MAKE) clean
+ rm -f build-arch-stamp build-indep-stamp configure-stamp msg #CONFIGURE-STAMP#
+ [ ! -f Makefile ] || $(MAKE) distclean
dh_clean
install: install-indep install-arch
only in patch2:
unchanged:
--- msort-8.52.orig/dstr.c
+++ msort-8.52/dstr.c
@@ -79,7 +79,7 @@
#ifdef SAFECALL
if(length == 0) return(ERROR);
#endif
- if (length > tgt->c) {
+ if (length + 1 > tgt->c) {
if(tgt->s != NULL) free( (void *) tgt->s);
tgt->s = (wchar_t *) malloc((length + 1) * sizeof(wchar_t));
if(tgt->s == NULL) return(ERROR);
