Ticket #3131 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

The use of env as a global variable in dbhelper.py looks dangerous

Reported by: jonas@edgewall.com Assigned to: bobbysmith007
Priority: highest Component: TimingAndEstimationPlugin
Severity: critical Keywords:
Cc: Trac Release: 0.11

Description

Unless I'm missing something the way dbhelper.py stores the env in a global variable is really unsafe. With thread based frontends like tracd or fastcgi this global variable can at anytime be modified by another thread to point to another environment. And this can cause the wrong database to be modified.

Attachments

Change History

06/04/08 00:00:20 changed by bobbysmith007

I hadnt realized that when I wrote it. I will attempt to fix this very soon.

06/04/08 00:24:51 changed by bobbysmith007

  • priority changed from normal to highest.
  • severity changed from normal to critical.

I have this patched, but am going to wait till I can test everything in the morning before publishing this.

Thanks again,

Russ

06/04/08 15:30:59 changed by bobbysmith007

  • status changed from new to closed.
  • resolution set to fixed.

(In [3784]) closes #3131
closes #3074
T&E 0.6.6

Fixes horrible bug relating to database access (due to my misunderstanding of which things were stateful in trac/python).

Fixed some bugs in the way billable values were being compared in the reports file (strong vs week typed databases)

06/04/08 15:36:01 changed by bobbysmith007

(In [3785]) closes #3131
closes #3074
T&E(trac 10) 0.6.6

I brought the trac 10 branch up to the same version number of the plugin so that hopefully there is less confusion on my part about versions.

Fixes horrible bug relating to database access (due to my misunderstanding of which things were stateful in trac/python).

Fixed some bugs in the way billable values were being compared in the reports file (strong vs weak typed databases)

Add/Change #3131 (The use of env as a global variable in dbhelper.py looks dangerous)




Change Properties
Action