public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* [Bug translator/5376] New: Stap does not check incorrect probe point syntax
@ 2007-11-21  0:21 mhiramat at redhat dot com
  2007-11-27 23:04 ` [Bug translator/5376] " mhiramat at redhat dot com
  2008-11-28 17:08 ` fche at redhat dot com
  0 siblings, 2 replies; 3+ messages in thread
From: mhiramat at redhat dot com @ 2007-11-21  0:21 UTC (permalink / raw)
  To: systemtap

- 'probe b* g* n {}' is recognized as 'b*g*n'

- 'probe hoge* if(e) {}' is recognized as 'hoge*if()' (concatenate 'hoge*' and 'if')

- 'probe huga? = hoge {}' is recognized as 'huga = hoge' (? is just ignored)

I think all those cases should be treated as incorrect syntaxes.

Thanks,

-- 
           Summary: Stap does not check incorrect probe point syntax
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: translator
        AssignedTo: systemtap at sources dot redhat dot com
        ReportedBy: mhiramat at redhat dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=5376

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug translator/5376] Stap does not check incorrect probe point syntax
  2007-11-21  0:21 [Bug translator/5376] New: Stap does not check incorrect probe point syntax mhiramat at redhat dot com
@ 2007-11-27 23:04 ` mhiramat at redhat dot com
  2008-11-28 17:08 ` fche at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: mhiramat at redhat dot com @ 2007-11-27 23:04 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From mhiramat at redhat dot com  2007-11-27 23:04 -------
Created an attachment (id=2111)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=2111&action=view)
A bugfix patch for wildcard parsing.

This patch will fix the first and the second issue.

With this patch, parser treats '*'(wildcard) in probe points as an alphabet.
However, there is a side-effect that this patch allows consecutive wildcards
(ex. "probe a** {}").

I'm not sure why the consecutive wildcards have been restricted. I think
it could be allowed like as bash.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=5376

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug translator/5376] Stap does not check incorrect probe point syntax
  2007-11-21  0:21 [Bug translator/5376] New: Stap does not check incorrect probe point syntax mhiramat at redhat dot com
  2007-11-27 23:04 ` [Bug translator/5376] " mhiramat at redhat dot com
@ 2008-11-28 17:08 ` fche at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: fche at redhat dot com @ 2008-11-28 17:08 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From fche at redhat dot com  2008-11-28 17:07 -------
commit 1ad820e finally fixes issue 3.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=5376

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-11-28 17:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-11-21  0:21 [Bug translator/5376] New: Stap does not check incorrect probe point syntax mhiramat at redhat dot com
2007-11-27 23:04 ` [Bug translator/5376] " mhiramat at redhat dot com
2008-11-28 17:08 ` fche at redhat dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).