On Mon, 14 Apr 2014 12:33:03 +0200
g...@suckless.org wrote:
Simplify skipping . and .. in switch_root
A small note: Don't forget to include string.h when you're using
strcmp().
Cheers
FRIGN
--
FRIGN d...@frign.de
On Fri, 18 Apr 2014 12:37:22 +0200
g...@suckless.org wrote:
-main(int argc, char **argv)
+main(int argc, char *argv[])
Can someone give me a reason why one should be favored over the other?
I suppose this is just a matter of taste (FYI: I use char **argv).
Cheers
FRIGN
--
FRIGN d
On Sat, 31 May 2014 18:56:43 +0200
g...@suckless.org wrote:
Note, the code is largely c89 clean. This is to avoid certain
warnings on some systems (OSX).
Thanks! Now I can finally build sbase on my MacBook Air without
warnings.
Cheers
FRIGN
--
FRIGN d...@frign.de
On Sun, 01 Jun 2014 17:10:40 +0200
g...@suckless.org wrote:
By the recommendation of FRIGN I refactored xsetcolorname to remove the
unnecessary r, g, b variables when allocating a new color. Colors are
now freed and set to the new color. A die() should not happen here. Oth
On Tue, 03 Jun 2014 13:18:23 +0200
g...@suckless.org wrote:
Remove explicit_bzero() calls
I support removing these calls, but why not also in login.c?
Cheers
FRIGN
--
FRIGN d...@frign.de
On Wed, 04 Jun 2014 14:23:34 +0200
g...@suckless.org wrote:
Use the macros from inttypes.h to properly print uint64_t
This is really scary. I've just started porting all long long's to
uint_least64_t and now you're using exactly the same in a similar
context.
Cheers
FRIGN
--
FRIGN d
-philosophy.
Cheers
FRIGN
--
FRIGN d...@frign.de
there. A suckless implementation would be cool, given it's
used very frequently.
I am not sure if quark should support inetd though, as we would need
to support both inetd and the builtin networking code.
It wouldn't make much sense.
Cheers
FRIGN
--
FRIGN d...@frign.de
added the discussed functionality and am now running afl-fuzz on it
for pen-testing on 3 unicode-codepages with random ßß's in it with the
command-line:
$cat cut-tests/paper | ./cut -f 2-3 -d ßß
It looks good so far.
Cheers
FRIGN
--
FRIGN d...@frign.de
or fputs?. I usually don't check the return value of write
functions and at the end of the loop I do a call to ferror.
Calling ferror sounds like a nice idea. Let's take a look at it.
Cheers
FRIGN
--
FRIGN d...@frign.de
), but felt like
writing this mail for anybody who's interested. :)
Cheers
FRIGN
--
FRIGN d...@frign.de
=286df29e7d78ae4a6b4474bb14d517e02e4f0489
Cheers
FRIGN
--
FRIGN d...@frign.de
the length at hand is a good thing.
On the other hand, join is completed feature-wise, so why
not just store the length on the heap locally in the function
and be done with it?
It's not barbarous at all.
Cheers
FRIGN
--
FRIGN d...@frign.de
On Sat, 13 Jun 2015 10:06:04 +0530
Aditya Goturu aditya3...@gmail.com wrote:
Is there any particular reason why I would use unix's fd and open()
instead of ANSI's FILE struct and fopen()?
Yes.
--
FRIGN d...@frign.de
file is unsorted, the return value is 1, as expected.
For convenience reasons, check()'s warning now includes the filename.
As a quick note, thanks to Dimitris we found out that Solaris also does it like
that. We are not alone in this world.
--
FRIGN d...@frign.de
this direction are much appreciated.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
worry about it in png2ff and ff2png,
especially in regard to handling 64-Bit PNGs and other gems.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
you think about it!
Cheers
FRIGN
--
FRIGN d...@frign.de
From 584b615305589aa760d49e62cb7cf64d406d6653 Mon Sep 17 00:00:00 2001
From: FRIGN d...@frign.de
Date: Wed, 8 Jul 2015 17:00:16 +0200
Subject: [PATCH 1/5] Remove insane *_FILENO and EXIT_* usage
Any system having different assignments than
! This is a pretty
important tool. :)
I'll also take my time and comment on your patch adding the
t-flag.
Cheers
FRIGN
[0]: http://pubs.opengroup.org/onlinepubs/9699919799/utilities/od.html
--
FRIGN <d...@frign.de>
to be applied to mainline, where you want to make sure that
historic accuracy with specific commit-hashes is guaranteed.
In any other case, it doesn't bring much to the table.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
day, when I've taken
the exam.
On thing left to discuss would be how to handle the
printing. I'd heuristically set the format on the
next major integral type.
For instance, when we read in a 3 byte chunk, we
want to take the next larger integral type (which
is int32 with 4 bytes).
Cheers
FRIGN
--
ightly better style than setting the variable
directly at the declaration block.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
ocus one one single thing first, or you end up rewriting all your
patches anyway.
We first need to get the length-handling properly first.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Fri, 2 Oct 2015 03:07:00 -0400
Greg Reagle <greg.rea...@umbc.edu> wrote:
Hey Greg,
> Trying again.
keep in mind that these flags are XSI-extensions, which we generally
don't include in sbase.
Let's discuss this on IRC, given the code-changes are not that drastic.
Cheers
FRIGN
--
of these filenames is
always interpreted as a filename, an application could always specify the
address base format with the -A option."
Do you really want that? Let's not touch the forbidden fruit.
Cheers
FRIGN
[0]: http://pubs.opengroup.org/onlinepubs/9699919799/utilities/od.html
--
FRIGN <d...@frign.de>
On Tue, 8 Dec 2015 11:28:17 +0100
Quentin Rameau <quinq+hackers@fifth.space> wrote:
> We would try to free unitialized ff_row pointer in cleanup on jpeg
> errors.
thanks, applied! :)
--
FRIGN <d...@frign.de>
easons why sent doesn't
use PNG as an intermediate format.
So, what is the reason? What does netpbm really _do_ better than farbfeld?
And no, Roberto, writing an image in ed does not cut it.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
pply all the tools available in
> the netpbm suite on a bulk of images, and there are quite some of them.
I'll think about adding pbm2ff and ff2pbm to the farbfeld-utilities :P
Then the crowd is happy and you can use the netpbm suite.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
You've replaced the entire farbfeld parsing code in sent with the
boilerplate "offered" by netpbm. Good job!
A question for the diligent reader: Can you read in a netpbm file
without first looking into the docs?
Cheers
FRIGN
--
FRIGN <d...@frign.de>
re formatted, so that's how you can deduce a
ml-etiquette easily.
But feel free to send in a patch, so the newcomers know what it's about.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
your bloody mail.
--
FRIGN <d...@frign.de>
On Fri, 1 Jan 2016 14:19:29 +0100 (CET)
g...@suckless.org wrote:
> - die("tabbed-"VERSION", © 2009-2014"
> + die("tabbed-"VERSION", © 2009-2015"
You probably mean 2016.
--
FRIGN <d...@frign.de>
On Mon, 21 Dec 2015 19:05:17 +
Dimitris Papastamos <s...@2f30.org> wrote:
> uid_t, gid_t are typically unsigned, so the check needs to be
> == -1 otherwise it will never be true.
We both are wrong :P It can either be signed or unsigned.
--
FRIGN <d...@frign.de>
On Mon, 21 Dec 2015 19:37:10 +0100
Quentin Rameau <quinq@fifth.space> wrote:
> + if (uid == -1 && gid == -1)
> + usage();
better use < 0 than -1
--
FRIGN <d...@frign.de>
"[[-k def] ...] [file ...]\n", argv0);
Please make that a separate patch and change the manpage accordingly.
> diff --git a/tar.c b/tar.c
Ok.
> diff --git a/touch.c b/touch.c
Ok.
> diff --git a/tr.c b/tr.c
>
> static void
> usage(void)
> {
> - eprintf("usage: %s [-cCds] set1 [set2]\n", argv0);
> + eprintf("usage: %s [-Ccds] set1 [set2]\n", argv0);
Same here. Change the manpage.
> diff --git a/xargs.c b/xargs.c
Ok.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Mon, 21 Dec 2015 18:36:28 +0100
Quentin Rameau <quinq@fifth.space> wrote:
> Some tools didn't use argv0 for tool name, or usage() at all.
Thanks, applied! :)
http://git.2f30.org/sbase/commit/?id=fd70f2afbeb1cacc0d55aa47831c84f39ec80978
--
FRIGN <d...@frign.de>
m
while working with the actual data.
I again want to stress here that I've been dealing with libpng for
almost 2 days now and I've still not found a satisfying general way
to handle 16-Bit PNG's properly.
The interface can be compared to the 6th circle of hell.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
on
here about this. I don't care abour your opinion that
much anyway.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
Hello fellow hackers,
I'll drop this little patch here so we finally make the switch to the
safe OpenBSD-functions for string copying.
Read the patch description for more info.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
>From 849a7cbee0310beb7ea51986bf98aff8d3b7ff26 Mon Sep 17 00:00:00
I just saw that I forgot to amend this part of the patch to this
one here. Will push it later, like the following:
if (strlcpy(..., size) >= size)
...
The only reason why strlcpy is "non-portable" is because the
Posix-committee has sticks up their asses (same with the
gl
shit, thanks.
there is also another point here: strlcpy is safer than strcpy
and strncpy because _if_ there is an overflow the string will
be 0-terminated. I'm not sure if there even should be an
error-out in case for instance we overflow writing the
"broken"-state-string to a client-name.
Cheer
ot to drop before pushing.
> Sorry for the noise.
I was already wondering. If you had applied that change, you would've
also had to make the changes in the arch-code-lookup-tables.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
l. The idiom of making a pointer struct-array
was circumvented to avoid doing that.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
rbage.
However, at least this garbage won't blow up in your
face.
> PD: I don't want to begin a flame war, but please, stop
> of being a fan boy and think for yourself, try to find
> the strong points and what is propaganda.
Maybe you should evaluate your position a bit better
before going all grandpa here on this ml.
Your first code example is embarassing and shows that
despite your high skill in C you seem to be fighting
the wrong war here.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Mon, 13 Jun 2016 14:17:01 +0200 (CEST)
g...@suckless.org wrote:
> - 2006-2013 suckless.org community | href="http://garbe.us/Contact;>Impressum
> + 2006-2015 suckless.org community | href="http://garbe.us/Contact;>Impressum
Isn't it 2016?
--
FRIGN <d...@frign.de>
eeded, afaik there
is none.
Commercial law is pretty clear in this case. If
something blows your computer up, you could
theoretically sue. So yeah, let's keep it.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Sun, 17 Jan 2016 10:10:19 +
Dimitris Papastamos <s...@2f30.org> wrote:
> Commit message reads like a diary.
That's because it is a diary. :P
--
FRIGN <d...@frign.de>
On Sat, 25 Jun 2016 19:11:09 +0200
pranomostro <pranomes...@gmail.com> wrote:
Hey Pranomostro,
nice catch! Care to send it as a git-patch?
Cheers
FRIGN
> ---
> printf.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/printf.c b/printf.c
>
se estrlcat() instead.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Mon, 29 Feb 2016 15:34:42 +0100 (CET)
g...@suckless.org wrote:
> Add a number of man pages
I swear to you, you are the only guy I know who writes the manuals
before the code.
--
FRIGN <d...@frign.de>
This requires a strict tracking of string lengths.
A short comment here: Of course I'll import memmem from OpenBSD into libutil,
given it's a GNU extension.
--
FRIGN <d...@frign.de>
se arg.h.
We need argv0 here because I moved the prepending of the argv[0] into
die(), and thus we need to store the argv0 in a global variable.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
slock as second + further arguments.
However, I respect your stances on this and will revert it, but also document -h
in the manpage.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
to just pass argv[0] to each call of die.
But it is not.
Would your approach be to just set argv0 as argv[0] and carry on
without modifying argc, argv?
I was surprised this idiomatic one-liner has received such opposition.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
for your patch, but please resend it with git format-patch in a
response to this thread! :)
Cheers
FRIGN
--
FRIGN <d...@frign.de>
do is error out and say
"libpng does not support zero dimension images"
however, this would be a statement based on one current version of libpng,
and we cannot predict how the format will develop.
The png_set_iHDR-function will error out if we pass bad data to it, so why
bother in the
On Mon, 7 Mar 2016 14:08:04 +0200
Vasily Kolobkov <polezaivs...@openmailbox.org> wrote:
> - = 40⋅30⋅10⁴ + (40⋅20 + 30⋅10)⋅10² + 10⋅20, but the middle is
> + = 4⋅3⋅10⁴ + (40⋅20 + 30⋅10)⋅10² + 10⋅20, but the middle is
4000 * 3000 = 40 * 30 * 10^4 = 4 * 3 * 10^6.
--
FRIGN <d...@frign.de>
= 0 here and use the form
> without relational operator for logic or pointers values.
You know how it is with style, but I respect your opinion on this
matter as a senior programmer.
I was wondering, don't you mean "argc != 0"? :)
Cheers
FRIGN
--
FRIGN <d...@frign.de>
t
this highly depends on the libc you are using (and esp. the Regex engine).
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Wed, 30 Mar 2016 05:23:08 +0200
Mattias Andrée <maand...@kth.se> wrote:
> Unlike your usual grep -r, this implementation
> uses breadth-first search. It usually finds
> makes it find what you are looking for faster.
Try working with recurse() in libutil.
Cheers
FRIG
you could also try a
setlocale(LC_ALL, "UTF-8");
in grep(1). Maybe that works.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
ou are right. It's general convention that we don't use capital letters here.
That's why this patch is wrong.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
d in fread(3).
thank you very much for this patch! I've applied it. Very good work!
Cheers
FRIGN
--
FRIGN <d...@frign.de>
t):
ansic:18805 (97.57%)
awk:255 (1.32%)
sh: 213 (1.11%)
do you really think it's a good idea to increase its size
by roughly 10% to accomodate one single command?
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Tue, 26 Jul 2016 22:52:16 +0200 (CEST)
g...@suckless.org wrote:
> Print highlighted input text only on single match
You probably merged the wrong branch here mate.
--
FRIGN <d...@frign.de>
e \n and it's easy to miss given some
errors rarely happen.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
};
with the same effect.
Can you give me a good reason or context example where adding the
compression support to 2ff proves to be useful?
Cheers
FRIGN
[0]: http://git.suckless.org/sent
--
FRIGN <d...@frign.de>
On Thu, 14 Jul 2016 10:38:05 +0200
FRIGN <d...@frign.de> wrote:
> { "\\.ff.bz2$", "bunzip | 2ff" },
my bad, it has to be
{ "\\.ff.bz2$", "bunzip" },
--
FRIGN <d...@frign.de>
Hello everybody,
I wrote a patch as an addition to config.def.h to support ff and ff.bz2
images. Let me know what you think.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
>From c8504ac1a1f1f508fc0989bdc8d0a59c8efd2d98 Mon Sep 17 00:00:00 2001
From: FRIGN <d...@frign.de>
Date: Thu, 14
you bloody serious?
--
FRIGN <d...@frign.de>
On Sun, 7 Aug 2016 23:33:58 +0100
Dimitris Papastamos <s...@2f30.org> wrote:
> hackers@ is not the right place for this kind of discussion.
My bad, I'll repost this on dev@.
--
FRIGN <d...@frign.de>
implied
beauty[1]. Do I really need to dig around github now to see what the
commit fixed?
What do you think?
Cheers
FRIGN
[0]:http://git.suckless.org/sxiv/tree/
[1]:http://git.suckless.org/sxiv/commit/?id=53a72c7b657d9dc3347d9d68e0b9a00773efe732
--
FRIGN <d...@frign.de>
On Sun, 7 Aug 2016 13:03:48 +0200
Hiltjo Posthuma <hil...@codemadness.org> wrote:
> Can someone apply this patch or give a reason why it sucks? :)
I second this patch.
--
FRIGN <d...@frign.de>
y a bit.
But again, if it is your explicit desire, I can create a version of the
patch with the brackets removed.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
and
makes the code more readable by pulling in arg.h, which has proven to
be very useful in the last few years.
Please provide feedback on this patch and test it.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
>From 3c8552d65a0cd999e0f485669e5c99ea7c3399a4 Mon Sep 17 00:00:00 2001
From:
Oh yeah, good catch!
> > diff --git a/util.h b/util.h
> > index 6f748b8..4f170a2 100644
> > --- a/util.h
> > +++ b/util.h
> > @@ -1,2 +1,6 @@
> > +#include "arg.h"
> > +
> > +extern char *argv0;
> > +
> > #undef explicit_bzero
&g
iew, and I'm
confident that this is given here on our mailing list (as we can also
observe with my patch). :)
I corrected it accordingly, as you probably have already seen.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Mon, 22 Aug 2016 10:59:16 +0200
Anselm R Garbe <garb...@gmail.com> wrote:
> I prefer it this way.
So is it good to merge?
Cheers
FRIGN
--
FRIGN <d...@frign.de>
with a bogus "x" as password just
> to see if the pws value is correct and other system requirements are
> met to call crypt later on after the password has been entered.
>
> I will apply it tomorrow if there are no objections.
are you sure we are not hitting any TOCTTOU proble
sizeof(struct hw)
shows clearly it's a type, whereas
sizeof(homework)
is not clear. But this is more a criticism of extreme typedeffing. Stop
using typedefs for structs and you won't have this problem any more. If
you can't discern type names from variable names, this is your pr
r, if you cannot discern a type name from a
variable name it's your problem and should not be "fixed" by having a
strange use of sizeof which is error-prone, as Ali has well shown.
This obviously goes the other way around too. One should not use
variable names that can be misinterpreted as
t least make it visually half-appealing by "shortening" their
code with struct typedefs. If your code has become so complex and
overwhelming that plain "struct structname" constructs can't be fitted
in there any more you should fix it. That's my opinion.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
action" just like using
if () {
} else if () {
if () {
} else {
}
}
trees just not to have gotos in the code.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
r structs.
I don't see a good reason to use typedefs in dwm/dmenu in the first
place, but I respect the per-process style.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Mon, 5 Sep 2016 10:05:16 +0200
FRIGN <d...@frign.de> wrote:
> [...], but I respect the per-process style.
Of course I meant "per-project style".
--
FRIGN <d...@frign.de>
es, which is
not a very sane thing to do. If this patch is applied, I'll send my
patch for slock as well to do the privilege drop properly.
Currently, the privilege drop on Linux does not reset supplementary
groups; this is adressed in the upcoming patch as well.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
to wait indefinitely
after forcefully ungrabbing input; so we just wait for the user to
release his keys (if it takes 2 seconds or whatever, say Mr. Plinkett
uses his Linux box :P).
Cheers
FRIGN
--
FRIGN <d...@frign.de>
d of the four next commits fixes that.
> The other three try to move duplicated behaviour inside unique
> functions and move global variables inside functions which
> exclusively need them. Patches following.
the patches look good to me and I would favor them to be merged.
Thanks for your wo
$ tool -f farg arg
when the flag f takes a mandatory argument farg the
resulting argc for instance is 1 (and *argv points at
"arg").
I hope that explains things.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Wed, 7 Sep 2016 15:17:11 +0200
FRIGN <d...@frign.de> wrote:
>
Okay, this is hopefully the last iteration.
--
FRIGN <d...@frign.de>
>From e308e34f49c89612ecdd17e989483c211453b6cb Mon Sep 17 00:00:00 2001
From: FRIGN <d...@frign.de>
Date: Wed, 7 Sep 2016 13:32:29 +0200
On Wed, 7 Sep 2016 15:15:03 +0200
FRIGN <d...@frign.de> wrote:
>
Forget this patch, I forgot to remove my 1337 debugging line
system("id");
return 0;
from it. See attached for the fixed version. Sorry for the noise.
Cheers
FRIGN
--
FRIGN &l
the supplementary
groups.
There really should not be any security compromises with a setuid
application like slock.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
>From dbb170d18380a4c20c5a074e68a4f481f17d3d5e Mon Sep 17 00:00:00 2001
From: FRIGN <d...@frign.de>
Date: Wed, 7 Sep 2016 13:32:29 +0200
On Fri, 02 Sep 2016 17:26:12 +0300
"Ali H. Fardan" <r...@firemail.cc> wrote:
> sorry, wrong patch
Thanks! :)
http://git.suckless.org/quark/commit/?id=12ebbc5dedfb212af3b93bb1762aec31bb59562f
--
FRIGN <d...@frign.de>
not "necessary" to have parentheses for variable sizeof's.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
h. We intensely discussed this on IRC and it is the
only sane way to make sure the behaviour is proper.
Timing heuristics are never good.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
lock: unable to grab mouse pointer for
> screen %s\n",
> + screen);
> …
> + fprintf(stderr,
> + "slock: unable to grab keyboard for screen
> %s\n",
> + screen);
this looks like a good compromise. :)
Cheers
FRIGN
--
FRIGN <d...@frign.de>
nogroup.
It's also been the standard value for quark since forever.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
On Wed, 7 Sep 2016 15:25:56 +0200
FRIGN <d...@frign.de> wrote:
>
Quentin found a problem on OpenBSD which was due to the negligient use
of the passwd struct pointer. Given it points to static memory, we
ended up with a "different" struct when we went to se the privilege
dr
. The largest one is the one removing global state
of the program to make code audits simpler.
Cheers
FRIGN
--
FRIGN <d...@frign.de>
>From 86c4a4edc72958461c9142d218ac0a285751e708 Mon Sep 17 00:00:00 2001
From: FRIGN <d...@frign.de>
Date: Sun, 11 Sep 2016 23:08:19 +0200
Subj
On Sun, 11 Sep 2016 23:24:20 +0200
FRIGN <d...@frign.de> wrote:
> See attached. Most important is the patch which removes the
> abomination of user $USER which actually poses quite a risk and only
> is done on part of the systems.
So you can test this, do the following
1 - 100 of 112 matches
Mail list logo