<feed xmlns='http://www.w3.org/2005/Atom'>
<title>plan9port/src/libsec, branch patch</title>
<subtitle>Plan 9 from User Space (with patches)</subtitle>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/'/>
<entry>
<title>all: remove $OBJTYPE from build</title>
<updated>2020-12-30T12:53:28+00:00</updated>
<author>
<name>Russ Cox</name>
<email>rsc@swtch.com</email>
</author>
<published>2020-12-30T12:39:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=99dee78c2d44641ba56e5bb640d732f993b3dfa1'/>
<id>99dee78c2d44641ba56e5bb640d732f993b3dfa1</id>
<content type='text'>
Now that we assume pthreads, the only assembly
left is in libmp and libsec.
We only ever added assembly for 386.
The portable C code is fine for plan9port.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Now that we assume pthreads, the only assembly
left is in libmp and libsec.
We only ever added assembly for 386.
The portable C code is fine for plan9port.
</pre>
</div>
</content>
</entry>
<entry>
<title>Trivial changes: whitespace and modes.</title>
<updated>2020-01-10T14:54:30+00:00</updated>
<author>
<name>Dan Cross</name>
<email>cross@gajendra.net</email>
</author>
<published>2020-01-10T14:44:21+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=fa325e9b42b0bdfb48857d1958d9fb7ceac55151'/>
<id>fa325e9b42b0bdfb48857d1958d9fb7ceac55151</id>
<content type='text'>
Remote whitespace at the ends of lines.
Remove blank lines from the ends of files.
Change modes on source files so that they
are not executable.

Signed-off-by: Dan Cross &lt;cross@gajendra.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remote whitespace at the ends of lines.
Remove blank lines from the ends of files.
Change modes on source files so that they
are not executable.

Signed-off-by: Dan Cross &lt;cross@gajendra.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>all: remove .cvsignore files</title>
<updated>2017-09-14T15:25:17+00:00</updated>
<author>
<name>Kare Nuorteva</name>
<email>kare.nuorteva@me.com</email>
</author>
<published>2017-09-10T10:26:57+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=b3336e5cbb82042dfeedab8615022410b74f8520'/>
<id>b3336e5cbb82042dfeedab8615022410b74f8520</id>
<content type='text'>
Rename following .cvsignore files to .gitkeep since they are
required by the build (directories must exist before build):
	- bin/fossil/.gitkeep
	- bin/fs/.gitkeep
	- bin/venti/.gitkeep

Change-Id: I9c2865058480cffb3a4613f25e2eca1f7e5578c0
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Rename following .cvsignore files to .gitkeep since they are
required by the build (directories must exist before build):
	- bin/fossil/.gitkeep
	- bin/fs/.gitkeep
	- bin/venti/.gitkeep

Change-Id: I9c2865058480cffb3a4613f25e2eca1f7e5578c0
</pre>
</div>
</content>
</entry>
<entry>
<title>all: linux/arm64 port</title>
<updated>2015-06-03T15:34:38+00:00</updated>
<author>
<name>Aram Hăvărneanu</name>
<email>aram@mgk.ro</email>
</author>
<published>2014-11-17T16:55:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=3f1c876e14e7eb45c1e17e1f3f4a28d98ff9b35f'/>
<id>3f1c876e14e7eb45c1e17e1f3f4a28d98ff9b35f</id>
<content type='text'>
Really trivial port.

Change-Id: Ib2e3ad48df555ca4bd9339ddf79a832be895bd95
Reviewed-on: https://plan9port-review.googlesource.com/1030
Reviewed-by: Russ Cox &lt;rsc@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Really trivial port.

Change-Id: Ib2e3ad48df555ca4bd9339ddf79a832be895bd95
Reviewed-on: https://plan9port-review.googlesource.com/1030
Reviewed-by: Russ Cox &lt;rsc@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>all: linux/mips port</title>
<updated>2015-06-03T15:34:06+00:00</updated>
<author>
<name>Aram Hăvărneanu</name>
<email>aram@mgk.ro</email>
</author>
<published>2015-03-19T13:13:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=37d81a2915cfd2766f35847e291ba50a98ddb5f3'/>
<id>37d81a2915cfd2766f35847e291ba50a98ddb5f3</id>
<content type='text'>
Change-Id: I0af67bc44c7bb62a8e5a47eac597367f521f1c11
Reviewed-on: https://plan9port-review.googlesource.com/1210
Reviewed-by: Russ Cox &lt;rsc@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I0af67bc44c7bb62a8e5a47eac597367f521f1c11
Reviewed-on: https://plan9port-review.googlesource.com/1210
Reviewed-by: Russ Cox &lt;rsc@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>libsec: avoid undefined C</title>
<updated>2013-03-19T18:35:16+00:00</updated>
<author>
<name>Xi Wang</name>
<email>xi.wang@gmail.com</email>
</author>
<published>2013-03-19T18:35:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=8a2a5b8f2568a665f00741994c1247f0f7d3dffe'/>
<id>8a2a5b8f2568a665f00741994c1247f0f7d3dffe</id>
<content type='text'>
gcc compiles `p + length &lt; p' into 'length &lt; 0' since pointer overflow is undefined behavior in C.  This breaks the check against a large `length'.

Use `length &gt; pend - p' instead.

There's no need to check `length &lt; 0' since `length' is from length_decode() and should be non-negative.

===

Try the simplified code.

void bar(void);
void foo(unsigned char *p, int length)
{
        if (p + length &lt; p)
                bar();
}

$ gcc -S -o - t.c -O2
...
foo:
.LFB0:
        .cfi_startproc
        testl	%esi, %esi
        js	.L4
        rep
        ret
.L4:
        jmp	bar
        .cfi_endproc

Clearly `p' is not used at all.

R=rsc
CC=plan9port.codebot
https://codereview.appspot.com/7231069
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
gcc compiles `p + length &lt; p' into 'length &lt; 0' since pointer overflow is undefined behavior in C.  This breaks the check against a large `length'.

Use `length &gt; pend - p' instead.

There's no need to check `length &lt; 0' since `length' is from length_decode() and should be non-negative.

===

Try the simplified code.

void bar(void);
void foo(unsigned char *p, int length)
{
        if (p + length &lt; p)
                bar();
}

$ gcc -S -o - t.c -O2
...
foo:
.LFB0:
        .cfi_startproc
        testl	%esi, %esi
        js	.L4
        rep
        ret
.L4:
        jmp	bar
        .cfi_endproc

Clearly `p' is not used at all.

R=rsc
CC=plan9port.codebot
https://codereview.appspot.com/7231069
</pre>
</div>
</content>
</entry>
<entry>
<title>fix clang warnings reported by Tuncer Ayaz</title>
<updated>2012-10-21T15:25:08+00:00</updated>
<author>
<name>Russ Cox</name>
<email>rsc@swtch.com</email>
</author>
<published>2012-10-21T15:25:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=0cfb376070a4bef7b4168a9795e025437e1be79f'/>
<id>0cfb376070a4bef7b4168a9795e025437e1be79f</id>
<content type='text'>
R=rsc
http://codereview.appspot.com/6744054
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
R=rsc
http://codereview.appspot.com/6744054
</pre>
</div>
</content>
</entry>
<entry>
<title>libsec: fix dsagen 64-bit bug</title>
<updated>2011-10-05T20:23:01+00:00</updated>
<author>
<name>Russ Cox</name>
<email>rsc@swtch.com</email>
</author>
<published>2011-10-05T20:23:01+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=c38ef9e7b60d5a3cab522dc1fe3c7b97e77646ea'/>
<id>c38ef9e7b60d5a3cab522dc1fe3c7b97e77646ea</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>arm: build fixes</title>
<updated>2011-05-18T17:20:56+00:00</updated>
<author>
<name>Richard Miller</name>
<email>millerresearch@gmail.com</email>
</author>
<published>2011-05-18T17:20:56+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=9cf92718e9f2e82c7d1f2a896b5c93b01f1ad073'/>
<id>9cf92718e9f2e82c7d1f2a896b5c93b01f1ad073</id>
<content type='text'>
Get p9p to build cleanly for ARM - based on suggestions from rsc and eekee, plus a final tweak from me.  Tested (a bit) on sheevaplug (ubuntu jaunty), Nokia N800 (maemo 4) and Nokia N900 (maemo 5).

R=rsc
CC=plan9port.codebot
http://codereview.appspot.com/4524056
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Get p9p to build cleanly for ARM - based on suggestions from rsc and eekee, plus a final tweak from me.  Tested (a bit) on sheevaplug (ubuntu jaunty), Nokia N800 (maemo 4) and Nokia N900 (maemo 5).

R=rsc
CC=plan9port.codebot
http://codereview.appspot.com/4524056
</pre>
</div>
</content>
</entry>
<entry>
<title>SunOS fixes (thanks Aram Hăvărneanu)</title>
<updated>2011-05-17T22:48:42+00:00</updated>
<author>
<name>Russ Cox</name>
<email>rsc@swtch.com</email>
</author>
<published>2011-05-17T22:48:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rctt.net/plan9port/commit/?id=af0dea45319aaa8aad011feac472c367165e9020'/>
<id>af0dea45319aaa8aad011feac472c367165e9020</id>
<content type='text'>
R=rsc, rsc
http://codereview.appspot.com/4271084
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
R=rsc, rsc
http://codereview.appspot.com/4271084
</pre>
</div>
</content>
</entry>
</feed>
