<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/cpython-git.git/Python/marshal.c, branch zooba-patch-1</title>
<subtitle>github.com: python/cpython.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/'/>
<entry>
<title>fix marshal uninitialized variable warnings (#4114)</title>
<updated>2017-10-25T06:09:55+00:00</updated>
<author>
<name>Benjamin Peterson</name>
<email>benjamin@python.org</email>
</author>
<published>2017-10-25T06:09:55+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=88d5e2c938b014885044f1cfd016e62798f07fd2'/>
<id>88d5e2c938b014885044f1cfd016e62798f07fd2</id>
<content type='text'>
GCC says:
../cpython/Python/marshal.c: In function ‘PyMarshal_WriteLongToFile’:
../cpython/Python/marshal.c:70:35: warning: ‘wf.ptr’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                       else if ((p)-&gt;ptr != (p)-&gt;end) *(p)-&gt;ptr++ = (c); \
                                   ^~
../cpython/Python/marshal.c:70:47: warning: ‘wf.end’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                       else if ((p)-&gt;ptr != (p)-&gt;end) *(p)-&gt;ptr++ = (c); \
                                               ^~
../cpython/Python/marshal.c:77:10: warning: ‘wf.str’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     if (p-&gt;str == NULL)
         ~^~~~~

This isn't a real problem because if the file pointer is not NULL, the
string-related fields are never touched. But, it doesn't hurt to set the unused
fields to NULL.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
GCC says:
../cpython/Python/marshal.c: In function ‘PyMarshal_WriteLongToFile’:
../cpython/Python/marshal.c:70:35: warning: ‘wf.ptr’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                       else if ((p)-&gt;ptr != (p)-&gt;end) *(p)-&gt;ptr++ = (c); \
                                   ^~
../cpython/Python/marshal.c:70:47: warning: ‘wf.end’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                       else if ((p)-&gt;ptr != (p)-&gt;end) *(p)-&gt;ptr++ = (c); \
                                               ^~
../cpython/Python/marshal.c:77:10: warning: ‘wf.str’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     if (p-&gt;str == NULL)
         ~^~~~~

This isn't a real problem because if the file pointer is not NULL, the
string-related fields are never touched. But, it doesn't hurt to set the unused
fields to NULL.</pre>
</div>
</content>
</entry>
<entry>
<title>reduce marshal stack size in debug mode on windows (closes #27019)</title>
<updated>2016-07-07T06:37:02+00:00</updated>
<author>
<name>Benjamin Peterson</name>
<email>benjamin@python.org</email>
</author>
<published>2016-07-07T06:37:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=64c82753de70f312c80797f63b6b8b98232b6f0f'/>
<id>64c82753de70f312c80797f63b6b8b98232b6f0f</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix reference leaks introduced by the patch for issue #5308.</title>
<updated>2013-07-11T16:14:07+00:00</updated>
<author>
<name>Serhiy Storchaka</name>
<email>storchaka@gmail.com</email>
</author>
<published>2013-07-11T16:14:07+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=fe2e8395651beb4f8968562f6a4fc1f2a2465e1f'/>
<id>fe2e8395651beb4f8968562f6a4fc1f2a2465e1f</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Issue #5308: Raise ValueError when marshalling too large object (a sequence</title>
<updated>2013-02-13T10:07:43+00:00</updated>
<author>
<name>Serhiy Storchaka</name>
<email>storchaka@gmail.com</email>
</author>
<published>2013-02-13T10:07:43+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=34fe1b7a3db61ee1b92f495488b08532b6126450'/>
<id>34fe1b7a3db61ee1b92f495488b08532b6126450</id>
<content type='text'>
with size &gt;= 2**31), instead of producing illegal marshal data.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
with size &gt;= 2**31), instead of producing illegal marshal data.
</pre>
</div>
</content>
</entry>
<entry>
<title>Untabify C files. Will watch buildbots.</title>
<updated>2010-05-09T14:46:46+00:00</updated>
<author>
<name>Antoine Pitrou</name>
<email>solipsis@pitrou.net</email>
</author>
<published>2010-05-09T14:46:46+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=c83ea137d7e717f764e2f31fc2544f522de7d857'/>
<id>c83ea137d7e717f764e2f31fc2544f522de7d857</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Issue #7332: Remove the 16KB stack-based buffer in</title>
<updated>2010-04-21T22:53:29+00:00</updated>
<author>
<name>Antoine Pitrou</name>
<email>solipsis@pitrou.net</email>
</author>
<published>2010-04-21T22:53:29+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=18e63fbe64dc0c8dfd2b04bae1687f68c8d780bb'/>
<id>18e63fbe64dc0c8dfd2b04bae1687f68c8d780bb</id>
<content type='text'>
PyMarshal_ReadLastObjectFromFile, which doesn't bring any noticeable
benefit compared to the dynamic memory allocation fallback.  Patch by
Charles-François Natali.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
PyMarshal_ReadLastObjectFromFile, which doesn't bring any noticeable
benefit compared to the dynamic memory allocation fallback.  Patch by
Charles-François Natali.
</pre>
</div>
</content>
</entry>
<entry>
<title>more _PyString_Resize error checking</title>
<updated>2010-04-03T00:57:33+00:00</updated>
<author>
<name>Benjamin Peterson</name>
<email>benjamin@python.org</email>
</author>
<published>2010-04-03T00:57:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=bea424af98468a7fc5c505c949218e22862d2b40'/>
<id>bea424af98468a7fc5c505c949218e22862d2b40</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Set retval on PyOS_string_to_double failure.</title>
<updated>2009-10-31T12:47:47+00:00</updated>
<author>
<name>Mark Dickinson</name>
<email>dickinsm@gmail.com</email>
</author>
<published>2009-10-31T12:47:47+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=51ae4921b30ab48bd31976fc4f92345b1bd26f50'/>
<id>51ae4921b30ab48bd31976fc4f92345b1bd26f50</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Removed PyOS_ascii_atof from marshal.c, as mentioned in issue 7117. Also brings it more in line with py3k.</title>
<updated>2009-10-27T19:42:57+00:00</updated>
<author>
<name>Eric Smith</name>
<email>eric@trueblade.com</email>
</author>
<published>2009-10-27T19:42:57+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=b218d289e6f5ceeb7979528d5c10bfcfb378b0b6'/>
<id>b218d289e6f5ceeb7979528d5c10bfcfb378b0b6</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Removed calls to PyFloat_AsReprString.</title>
<updated>2009-10-19T00:34:12+00:00</updated>
<author>
<name>Eric Smith</name>
<email>eric@trueblade.com</email>
</author>
<published>2009-10-19T00:34:12+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/cpython-git.git/commit/?id=156692752db98a2e31c4e00f59072b9f221eb9a9'/>
<id>156692752db98a2e31c4e00f59072b9f221eb9a9</id>
<content type='text'>
This is in anticipation of possibly implementing issue 7117 (short float repr).
This removes the last calls to PyFloat_AsString, PyFloat_AsReprString, and
PyFloat_AsStringEx, which are unsafe.

Also, switch to defines for error values to bring this code more in line
with the py3k branch.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is in anticipation of possibly implementing issue 7117 (short float repr).
This removes the last calls to PyFloat_AsString, PyFloat_AsReprString, and
PyFloat_AsStringEx, which are unsafe.

Also, switch to defines for error values to bring this code more in line
with the py3k branch.
</pre>
</div>
</content>
</entry>
</feed>
