`\@currentlabelname` is not correctly sanitized

[u-fischer]

titlesec defines \@currentlabelname in \ttl@straight@i but doesn't sanitized the argument.

\def\ttl@straight@i#1[#2]#3{%
  \def\@currentlabelname{#2}% for nameref
  \gdef\ttl@savemark{\csname#1mark\endcsname{#3}}%
  \let\ttl@savewrite\@empty
  \def\ttl@savetitle{#3}%
  \gdef\thetitle{\csname the#1\endcsname}%
  \if@noskipsec \leavevmode \fi
  \par
  \ttl@labelling{#1}{#2}%
  \ttl@startargs\ttl@straight@ii{#1}{#3}}

If nameref is loaded this more or less works, but without it it breaks in a current LaTeX which now stores the labelname too:

\documentclass{article}
\usepackage{titlesec}
\begin{document}
\section{a\label{b}}
\end{document}

gives

! TeX capacity exceeded, sorry [input stack size=10000].
\protect ->\noexpand \protect 
                              \noexpand 
l.217 \section{a\label{b}}

titlesec could load the gettitlestring package and then do:

\GetTitleString{#2}%
\let\@currentlabelname\GetTitleStringResult

[jbezos]

I’m reopening this issue because it won’t get fixed in the short or medium term.

[davidcarlisle]

@u-fischer this is possibly enough, without introducing a new package dependency?

\documentclass{article}

\usepackage{titlesec}

\makeatletter
\def\removelabel#1\label#2#3\label#4\relax{#1#3}
\def\ttl@straight@i#1[#2]#3{%
  \expandafter\def\expandafter\@currentlabelname\expandafter{%
         \removelabel#2\label{}\label{}\relax}% for nameref
  \gdef\ttl@savemark{\csname#1mark\endcsname{#3}}%
  \let\ttl@savewrite\@empty
  \def\ttl@savetitle{#3}%
  \gdef\thetitle{\csname the#1\endcsname}%
  \if@noskipsec \leavevmode \fi
  \par
  \ttl@labelling{#1}{#2}%
  \ttl@startargs\ttl@straight@ii{#1}{#3}}
\makeatother
\begin{document}

\section{a\label{b}}

\end{document}

[u-fischer]

It would fix the label case, but generally I think it would be better if not every package defines its own sanitizing variant.

[davidcarlisle]

@u-fischer hmm we could also fix this in the kernel by copying the normalization used by \addtocontents to \label

\documentclass{article}

\usepackage{titlesec}

\makeatletter
\def\label#1{\@bsphack
  \begingroup
  \UseHookWithArguments{label}{1}{#1}%
  \protected@write\@auxout{%
    \let\label\@gobble \let\index\@gobble \let\glossary\@gobble% new
  }%
         {\string\newlabel{#1}{{\@currentlabel}{\thepage}%
          {\@currentlabelname}{\@currentHref}{\@kernel@reserved@label@data}}}%
  \endgroup
  \@esphack}
\makeatother
\begin{document}

\section{a\label{b}}

\end{document}

[u-fischer]

@davidcarlisle that is a good idea.