Artifact
3d071d49fa953dacac3c65d84236a3e6954099b5:
Ticket change
[3d071d49fa]
- Ticket
[6dba56543b] fossil ticket --quote segfaults when adding to the comment status still Open with 1 other change
by
drh
2012-08-08 11:21:37.
0000: 44 20 32 30 31 32 2d 30 38 2d 30 38 54 31 31 3a D 2012-08-08T11:
0010: 32 31 3a 33 37 2e 31 36 32 0a 4a 20 63 6f 6d 6d 21:37.162.J comm
0020: 65 6e 74 20 3c 70 72 65 3e 5c 72 5c 6e 24 5c 73 ent <pre>\r\n$\s
0030: 66 6f 73 73 69 6c 5c 73 6e 65 77 5c 73 66 6f 6f fossil\snew\sfoo
0040: 2e 66 6f 73 73 69 6c 5c 72 5c 6e 2e 2e 2e 5c 72 .fossil\r\n...\r
0050: 5c 6e 24 5c 73 66 6f 73 73 69 6c 5c 73 74 69 63 \n$\sfossil\stic
0060: 6b 65 74 5c 73 2d 52 5c 73 66 6f 6f 2e 66 6f 73 ket\s-R\sfoo.fos
0070: 73 69 6c 5c 73 61 64 64 5c 73 74 69 74 6c 65 5c sil\sadd\stitle\
0080: 73 74 65 73 74 5c 72 5c 6e 74 69 63 6b 65 74 5c stest\r\nticket\
0090: 73 61 64 64 5c 73 73 75 63 63 65 65 64 65 64 5c sadd\ssucceeded\
00a0: 73 66 6f 72 5c 73 55 49 44 5c 73 61 30 65 37 61 sfor\sUID\sa0e7a
00b0: 66 65 35 36 63 65 61 66 32 65 32 38 64 66 61 37 fe56ceaf2e28dfa7
00c0: 39 62 63 31 64 62 33 34 61 61 32 61 37 36 37 65 9bc1db34aa2a767e
00d0: 37 63 31 5c 72 5c 6e 24 5c 73 66 6f 73 73 69 6c 7c1\r\n$\sfossil
00e0: 5c 73 74 69 63 6b 65 74 5c 73 2d 52 5c 73 66 6f \sticket\s-R\sfo
00f0: 6f 2e 66 6f 73 73 69 6c 5c 73 63 68 61 6e 67 65 o.fossil\schange
0100: 5c 73 61 30 65 37 61 66 65 35 36 5c 73 63 6f 6d \sa0e7afe56\scom
0110: 6d 65 6e 74 5c 73 27 66 6f 6f 27 5c 73 2d 2d 71 ment\s'foo'\s--q
0120: 75 6f 74 65 5c 72 5c 6e 74 69 63 6b 65 74 5c 73 uote\r\nticket\s
0130: 73 65 74 5c 73 73 75 63 63 65 65 64 65 64 5c 73 set\ssucceeded\s
0140: 66 6f 72 5c 73 55 49 44 5c 73 61 30 65 37 61 66 for\sUID\sa0e7af
0150: 65 35 36 63 65 61 66 32 65 32 38 64 66 61 37 39 e56ceaf2e28dfa79
0160: 62 63 31 64 62 33 34 61 61 32 61 37 36 37 65 37 bc1db34aa2a767e7
0170: 63 31 5c 72 5c 6e 24 5c 73 66 6f 73 73 69 6c 5c c1\r\n$\sfossil\
0180: 73 74 69 63 6b 65 74 5c 73 2d 52 5c 73 66 6f 6f sticket\s-R\sfoo
0190: 2e 66 6f 73 73 69 6c 5c 73 63 68 61 6e 67 65 5c .fossil\schange\
01a0: 73 61 30 65 37 61 66 65 35 36 5c 73 2b 63 6f 6d sa0e7afe56\s+com
01b0: 6d 65 6e 74 5c 73 27 66 6f 6f 27 5c 73 2d 2d 71 ment\s'foo'\s--q
01c0: 75 6f 74 65 5c 72 5c 6e 73 65 67 6d 65 6e 74 61 uote\r\nsegmenta
01d0: 74 69 6f 6e 5c 73 66 61 75 6c 74 5c 72 5c 6e 3c tion\sfault\r\n<
01e0: 2f 70 72 65 3e 5c 72 5c 6e 5c 72 5c 6e 3c 68 72 /pre>\r\n\r\n<hr
01f0: 5c 73 2f 3e 3c 69 3e 6d 69 73 74 61 63 68 6b 69 \s/><i>mistachki
0200: 6e 5c 73 61 64 64 65 64 5c 73 6f 6e 5c 73 32 30 n\sadded\son\s20
0210: 31 32 2d 30 38 2d 30 38 5c 73 30 33 3a 34 37 3a 12-08-08\s03:47:
0220: 33 31 5c 73 55 54 43 3a 3c 2f 69 3e 3c 62 72 5c 31\sUTC:</i><br\
0230: 73 2f 3e 5c 72 5c 6e 52 65 61 64 69 6e 67 5c 73 s/>\r\nReading\s
0240: 74 68 65 5c 73 63 6f 64 65 2c 5c 73 49 27 6d 5c the\scode,\sI'm\
0250: 73 73 6f 6d 65 77 68 61 74 5c 73 63 6f 6e 66 75 ssomewhat\sconfu
0260: 73 65 64 5c 73 61 62 6f 75 74 5c 73 74 68 65 5c sed\sabout\sthe\
0270: 73 70 72 6f 70 6f 73 65 64 5c 73 66 69 78 2e 5c sproposed\sfix.\
0280: 73 5c 73 50 65 72 68 61 70 73 5c 73 69 74 5c 73 s\sPerhaps\sit\s
0290: 62 65 5c 73 73 6f 6d 65 74 68 69 6e 67 5c 73 6c be\ssomething\sl
02a0: 69 6b 65 5c 73 74 68 69 73 5c 73 69 6e 73 74 65 ike\sthis\sinste
02b0: 61 64 2c 5c 73 67 69 76 65 6e 5c 73 74 68 65 5c ad,\sgiven\sthe\
02c0: 73 6e 61 74 75 72 65 5c 73 6f 66 5c 73 74 68 65 snature\sof\sthe
02d0: 5c 73 6f 74 68 65 72 5c 73 3c 62 3e 69 66 3c 2f \sother\s<b>if</
02e0: 62 3e 5c 73 62 6c 6f 63 6b 5c 73 65 6e 63 6c 6f b>\sblock\senclo
02f0: 73 65 64 5c 73 77 69 74 68 5c 73 3c 62 3e 69 66 sed\swith\s<b>if
0300: 28 5c 73 74 6b 74 45 6e 63 6f 64 69 6e 67 5c 73 (\stktEncoding\s
0310: 3d 3d 5c 73 74 6b 74 46 6f 73 73 69 6c 69 7a 65 ==\stktFossilize
0320: 5c 73 29 3c 2f 62 3e 3f 5c 72 5c 6e 5c 72 5c 6e \s)</b>?\r\n\r\n
0330: 3c 76 65 72 62 61 74 69 6d 3e 5c 72 5c 6e 49 6e <verbatim>\r\nIn
0340: 64 65 78 3a 5c 73 73 72 63 2f 74 6b 74 2e 63 5c dex:\ssrc/tkt.c\
0350: 72 5c 6e 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d r\n=============
0360: 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d ================
0370: 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d ================
0380: 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d 3d ================
0390: 3d 3d 3d 3d 3d 5c 72 5c 6e 2d 2d 2d 5c 73 73 72 =====\r\n---\ssr
03a0: 63 2f 74 6b 74 2e 63 5c 72 5c 6e 2b 2b 2b 5c 73 c/tkt.c\r\n+++\s
03b0: 73 72 63 2f 74 6b 74 2e 63 5c 72 5c 6e 40 40 5c src/tkt.c\r\n@@\
03c0: 73 2d 31 31 36 37 2c 31 31 5c 73 2b 31 31 36 37 s-1167,11\s+1167
03d0: 2c 31 31 5c 73 40 40 5c 72 5c 6e 5c 73 5c 73 5c ,11\s@@\r\n\s\s\
03e0: 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 7d 65 6c s\s\s\s\s\s\s}el
03f0: 73 65 7b 5c 72 5c 6e 5c 73 5c 73 5c 73 5c 73 5c se{\r\n\s\s\s\s\
0400: 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 62 6c 6f s\s\s\s\s\s\sblo
0410: 62 5f 61 70 70 65 6e 64 66 28 26 74 6b 74 63 68 b_appendf(&tktch
0420: 6e 67 2c 5c 73 22 4a 25 73 25 73 5c 73 25 23 46 ng,\s"J%s%s\s%#F
0430: 5c 5c 6e 22 2c 5c 73 7a 50 66 78 2c 5c 72 5c 6e \\n",\szPfx,\r\n
0440: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 \s\s\s\s\s\s\s\s
0450: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 \s\s\s\s\s\s\s\s
0460: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 \s\s\s\s\s\s\s\s
0470: 61 7a 46 69 65 6c 64 5b 69 5d 2c 5c 73 73 74 72 azField[i],\sstr
0480: 6c 65 6e 28 7a 56 61 6c 75 65 29 2c 5c 73 7a 56 len(zValue),\szV
0490: 61 6c 75 65 29 3b 5c 72 5c 6e 5c 73 5c 73 5c 73 alue);\r\n\s\s\s
04a0: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 7d 5c 72 5c \s\s\s\s\s\s}\r\
04b0: 6e 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c n\s\s\s\s\s\s\s\
04c0: 73 5c 73 69 66 28 5c 73 74 6b 74 45 6e 63 6f 64 s\sif(\stktEncod
04d0: 69 6e 67 5c 73 3d 3d 5c 73 74 6b 74 46 6f 73 73 ing\s==\stktFoss
04e0: 69 6c 69 7a 65 5c 73 29 7b 5c 72 5c 6e 2d 5c 73 ilize\s){\r\n-\s
04f0: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 \s\s\s\s\s\s\s\s
0500: 5c 73 66 72 65 65 28 61 7a 56 61 6c 75 65 5b 69 \sfree(azValue[i
0510: 5d 29 3b 5c 72 5c 6e 2b 5c 73 5c 73 5c 73 5c 73 ]);\r\n+\s\s\s\s
0520: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 66 72 65 65 \s\s\s\s\s\sfree
0530: 28 7a 46 56 61 6c 75 65 29 3b 5c 72 5c 6e 5c 73 (zFValue);\r\n\s
0540: 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 5c 73 \s\s\s\s\s\s\s\s
0550: 7d 5c 72 5c 6e 5c 73 5c 73 5c 73 5c 73 5c 73 5c }\r\n\s\s\s\s\s\
0560: 73 5c 73 7d 5c 72 5c 6e 5c 73 5c 73 5c 73 5c 73 s\s}\r\n\s\s\s\s
0570: 5c 73 5c 73 5c 73 62 6c 6f 62 5f 61 70 70 65 6e \s\s\sblob_appen
0580: 64 66 28 26 74 6b 74 63 68 6e 67 2c 5c 73 22 4b df(&tktchng,\s"K
0590: 5c 73 25 73 5c 5c 6e 22 2c 5c 73 7a 54 6b 74 55 \s%s\\n",\szTktU
05a0: 75 69 64 29 3b 5c 72 5c 6e 5c 73 5c 73 5c 73 5c uid);\r\n\s\s\s\
05b0: 73 5c 73 5c 73 5c 73 62 6c 6f 62 5f 61 70 70 65 s\s\s\sblob_appe
05c0: 6e 64 66 28 26 74 6b 74 63 68 6e 67 2c 5c 73 22 ndf(&tktchng,\s"
05d0: 55 5c 73 25 46 5c 5c 6e 22 2c 5c 73 7a 55 73 65 U\s%F\\n",\szUse
05e0: 72 29 3b 5c 72 5c 6e 5c 73 5c 73 5c 73 5c 73 5c r);\r\n\s\s\s\s\
05f0: 73 5c 73 5c 73 6d 64 35 73 75 6d 5f 62 6c 6f 62 s\s\smd5sum_blob
0600: 28 26 74 6b 74 63 68 6e 67 2c 5c 73 26 63 6b 73 (&tktchng,\s&cks
0610: 75 6d 29 3b 5c 72 5c 6e 3c 2f 76 65 72 62 61 74 um);\r\n</verbat
0620: 69 6d 3e 5c 72 5c 6e 5c 72 5c 6e 5c 72 5c 6e 3c im>\r\n\r\n\r\n<
0630: 68 72 5c 73 2f 3e 3c 69 3e 72 6d 61 78 5c 73 61 hr\s/><i>rmax\sa
0640: 64 64 65 64 5c 73 6f 6e 5c 73 32 30 31 32 2d 30 dded\son\s2012-0
0650: 38 2d 30 38 5c 73 31 31 3a 31 38 3a 32 30 5c 73 8-08\s11:18:20\s
0660: 55 54 43 3a 3c 2f 69 3e 3c 62 72 5c 73 2f 3e 5c UTC:</i><br\s/>\
0670: 72 5c 6e 49 6e 64 65 65 64 2c 5c 73 74 68 65 5c r\nIndeed,\sthe\
0680: 73 64 69 73 74 69 6e 63 74 69 6f 6e 5c 73 77 68 sdistinction\swh
0690: 65 74 68 65 72 5c 73 69 74 5c 73 69 73 5c 73 61 ether\sit\sis\sa
06a0: 7a 41 70 70 65 6e 64 26 23 39 31 3b 69 26 23 39 zAppend[i	
06b0: 33 3b 5c 73 6f 72 5c 73 61 7a 56 61 6c 75 65 26 3;\sor\sazValue&
06c0: 23 39 31 3b 69 26 23 39 33 3b 5c 73 74 68 61 74 #91;i]\sthat
06d0: 5c 73 6e 65 65 64 73 5c 73 74 6f 5c 73 62 65 5c \sneeds\sto\sbe\
06e0: 73 66 72 65 65 64 5c 73 69 73 5c 73 61 6c 72 65 sfreed\sis\salre
06f0: 61 64 79 5c 73 62 65 69 6e 67 5c 73 6d 61 64 65 ady\sbeing\smade
0700: 5c 73 74 77 6f 5c 73 69 66 5c 73 62 6c 6f 63 6b \stwo\sif\sblock
0710: 73 5c 73 61 62 6f 76 65 2c 5c 73 73 6f 5c 73 77 s\sabove,\sso\sw
0720: 65 5c 73 64 6f 6e 27 74 5c 73 6e 65 65 64 5c 73 e\sdon't\sneed\s
0730: 74 6f 5c 73 72 65 70 65 61 74 5c 73 69 74 5c 73 to\srepeat\sit\s
0740: 68 65 72 65 5c 73 61 6e 64 5c 73 63 61 6e 5c 73 here\sand\scan\s
0750: 6a 75 73 74 5c 73 66 72 65 65 5c 73 7a 56 61 6c just\sfree\szVal
0760: 75 65 2e 0a 4b 20 36 64 62 61 35 36 35 34 33 62 ue..K 6dba56543b
0770: 38 39 64 63 34 31 34 36 33 31 37 34 63 36 33 63 89dc41463174c63c
0780: 39 66 31 31 65 37 39 38 63 39 63 66 62 34 0a 55 9f11e798c9cfb4.U
0790: 20 64 72 68 0a 5a 20 30 33 33 61 35 65 64 35 65 drh.Z 033a5ed5e
07a0: 32 37 62 33 34 32 36 36 35 37 64 36 35 62 38 63 27b3426657d65b8c
07b0: 62 38 32 64 61 32 34 0a b82da24.