NotificationsYou must be signed in to change notification settings
Fork8.1k
Star22.1k

Commit6af8ef0

committed

Fix a race condition in TexManager.make_dvi.

Previously, a race condition could occur if, while a process had calledmake_tex (generating the tex file in the global cache) and was going tocall the latex subprocess (to generate the dvi file), another processalso called make_tex for the same tex string and started rewriting thetex source. In that case, the latex subprocess could see a partiallywritten (invalid) tex source.Fix that by generating the tex source in a process-private temporarydirectory, where the latex process is already going to run anyways.(This is cheap compared to the latex subprocess invocation.)

1 parent780e66c commit6af8ef0Copy full SHA for 6af8ef0

File tree

1 file changed

-6

lines changed

lib/matplotlib
- texmanager.py

1 file changed

-6

lines changed

`‎lib/matplotlib/texmanager.py‎`

Lines changed: 8 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -284,8 +284,10 @@ def make_dvi(cls, tex, fontsize):`
`284`	`284`	`basefile=cls.get_basefile(tex,fontsize)`
`285`	`285`	`dvifile='%s.dvi'%basefile`
`286`	`286`	`ifnotos.path.exists(dvifile):`
`287`		`-texfile=Path(cls.make_tex(tex,fontsize))`
`288`		`-# Generate the dvi in a temporary directory to avoid race`
	`287`	`+# Also generate the tex source in the main cache directory, but`
	`288`	`+# only for backcompat.`
	`289`	`+cls.make_tex(tex,fontsize)`
	`290`	`+# Generate the tex and dvi in a temporary directory to avoid race`
`289`	`291`	`# conditions e.g. if multiple processes try to process the same tex`
`290`	`292`	`# string at the same time. Having tmpdir be a subdirectory of the`
`291`	`293`	`# final output dir ensures that they are on the same filesystem,`
`@@ -296,12 +298,12 @@ def make_dvi(cls, tex, fontsize):`
`296`	`298`	# will be blocked when `openin_any = p` in texmf.cnf).
`297`	`299`	`cwd=Path(dvifile).parent`
`298`	`300`	`withTemporaryDirectory(dir=cwd)astmpdir:`
`299`		`-tmppath=Path(tmpdir)`
	`301`	`+Path(tmpdir,"file.tex").write_text(`
	`302`	`+cls._get_tex_source(tex,fontsize),encoding='utf-8')`
`300`	`303`	`cls._run_checked_subprocess(`
`301`	`304`	`["latex","-interaction=nonstopmode","--halt-on-error",`
`302`		`-f"--output-directory={tmppath.name}",`
`303`		`-f"{texfile.name}"],tex,cwd=cwd)`
`304`		`- (tmppath/Path(dvifile).name).replace(dvifile)`
	`305`	`+"file.tex"],tex,cwd=tmpdir)`
	`306`	`+Path(tmpdir,"file.dvi").replace(dvifile)`
`305`	`307`	`returndvifile`
`306`	`308`
`307`	`309`	`@classmethod`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit6af8ef0

File tree

1 file changed

1 file changed

`‎lib/matplotlib/texmanager.py‎`

0 commit comments