Adam,
Nice article. According to BOL you should use sys.traces instead of fn_trace_getinfo. Here is the BOL quote:
The sys.traces catalog view contains the current running traces on the system. This view is intended as a replacement for the fn_trace_getinfo function.
Also if you disable this trace many of the reports available in SSMS will no longer work as they get much of their information from this trace.
Jack Corbett
Consultant - Straight Path Solutions
Check out these links on how to get faster and more accurate answers:
Forum Etiquette: How to post data/code on a forum to get the best help
Need an Answer? Actually, No ... You Need a Question