[jira] [Commented] (TS-1042) PATCH: correct debug message in FetchSM

2011-12-09 Thread Commented

[ 
https://issues.apache.org/jira/browse/TS-1042?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13166115#comment-13166115
 ] 

Igor Galić commented on TS-1042:


crawling through {{printf(3)}}, giving up, asking {{##C}} - I now know what 
{{printf(%*.*s, length, length, string);}} does!

Thank you again for the patch, I aplied it in r1212343

 PATCH: correct debug message in FetchSM
 ---

 Key: TS-1042
 URL: https://issues.apache.org/jira/browse/TS-1042
 Project: Traffic Server
  Issue Type: Bug
  Components: HTTP
Reporter: James Peach
Assignee: Igor Galić
Priority: Minor
 Fix For: 3.1.2

 Attachments: 0004-Fix-FetchSM-debugging-message.patch


 In the FetchSM module, there is a debug message that can walk off the end of 
 the buffer. This patch corrects that by limiting the printed string to the 
 known length.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (TS-1042) PATCH: correct debug message in FetchSM

2011-12-09 Thread Leif Hedstrom (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1042?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13166222#comment-13166222
 ] 

Leif Hedstrom commented on TS-1042:
---

Hmmm, why %*.*s, length, length) ? Everywhere else we just do %.*s, length, 
string)

 PATCH: correct debug message in FetchSM
 ---

 Key: TS-1042
 URL: https://issues.apache.org/jira/browse/TS-1042
 Project: Traffic Server
  Issue Type: Bug
  Components: HTTP
Reporter: James Peach
Assignee: Igor Galić
Priority: Minor
 Fix For: 3.1.2

 Attachments: 0004-Fix-FetchSM-debugging-message.patch


 In the FetchSM module, there is a debug message that can walk off the end of 
 the buffer. This patch corrects that by limiting the printed string to the 
 known length.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (TS-1042) PATCH: correct debug message in FetchSM

2011-12-09 Thread James Peach (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1042?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13166330#comment-13166330
 ] 

James Peach commented on TS-1042:
-

To be honest, it's mostly from habit. However %*.*s prints exactly the number 
of bytes whereas %.*s prints up to the number of bytes. In this particular 
case we know the number of bytes and we want to print all of them so %*.*s 
seems like the right choice. But %.*s will fix the bug just as well.

 PATCH: correct debug message in FetchSM
 ---

 Key: TS-1042
 URL: https://issues.apache.org/jira/browse/TS-1042
 Project: Traffic Server
  Issue Type: Bug
  Components: HTTP
Reporter: James Peach
Assignee: Igor Galić
Priority: Minor
 Fix For: 3.1.2

 Attachments: 0004-Fix-FetchSM-debugging-message.patch


 In the FetchSM module, there is a debug message that can walk off the end of 
 the buffer. This patch corrects that by limiting the printed string to the 
 known length.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira