diff --git a/BUGS b/BUGS index d5c6cf3871c50a66142a2f5bbf6bcdebb307719c..b36caa75f0bbfaf7e53b0d5fb3bdc2a6a7f89eab 100644 --- a/BUGS +++ b/BUGS @@ -94,11 +94,11 @@ browser..) (52) [FIXED] - Alt speller argument (-s, --speller) does not take a string argument of more than one word. (53) [FIXED]. +- Cut to end cutting (-k) causes segfaults (try cutting "- Backup making + (filename~)?" line in TODO file) (discovred by + higuita@cadernoverde.com) (54) [FIXED]. + ** Open BUGS ** $Id$ -- Cut to end cutting (-k) causes segfaults (try cutting "- Backup making - (filename~)?" line in TODO file) (discovred by - higuita@cadernoverde.com) (54). - diff --git a/ChangeLog b/ChangeLog index b7cc3d43e585fa8c221ae11974e8e26214221eef..49d535817a9729a32f9631b3e83689e02bff44f0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,6 +1,9 @@ CVS code - - configure.in: - Autoconf compatibility fixes (Pavel Roskin) +- cut.c: + do_cut_text() + - marked text cut fixes (Rocco) (Fixes bug #54). nano-0.9.99pre2 - 01/31/2001 General diff --git a/cut.c b/cut.c index 1d9d2b765aec5f9edb76c01df77c358c048f4460..5e8cba7569e01ced4d6df49a475e2fbdfed9b0ec 100644 --- a/cut.c +++ b/cut.c @@ -174,24 +174,23 @@ int do_cut_text(void) if (ISSET(MARK_ISSET)) { if (current->lineno == mark_beginbuf->lineno) { tmp = copy_node(current); - newsize = abs(strlen(¤t->data[mark_beginx]) - - strlen(¤t->data[current_x])); + newsize = abs(mark_beginx - current_x) + 1; - tmpstr = nmalloc(newsize); + tmpstr = nmalloc(newsize + 1); if (current_x < mark_beginx) { strncpy(tmpstr, ¤t->data[current_x], newsize); memmove(¤t->data[current_x], ¤t->data[mark_beginx], - strlen(¤t->data[mark_beginx] - newsize)); + strlen(¤t->data[mark_beginx]) + 1); } else { strncpy(tmpstr, ¤t->data[mark_beginx], newsize); memmove(¤t->data[mark_beginx], ¤t->data[current_x], - strlen(¤t->data[current_x] - newsize)); + strlen(¤t->data[current_x]) + 1); current_x = mark_beginx; update_cursor(); } - tmpstr[newsize] = 0; + tmpstr[newsize - 1] = 0; tmp->data = tmpstr; add_to_cutbuffer(tmp); dump_buffer(cutbuffer);